[PATCH] D43197: [OpenMP] Add flag for linking runtime bitcode library
Jonas Hahnfeld via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 9 01:21:04 PST 2018
Hahnfeld added inline comments.
================
Comment at: lib/Driver/ToolChains/Cuda.cpp:536-542
+ StringRef CompilerPath = env;
+ while (!CompilerPath.empty()) {
+ std::pair<StringRef, StringRef> Split =
+ CompilerPath.split(llvm::sys::EnvPathSeparator);
+ LibraryPaths.push_back(Split.first);
+ CompilerPath = Split.second;
+ }
----------------
gtbercea wrote:
> Hahnfeld wrote:
> > gtbercea wrote:
> > > Hahnfeld wrote:
> > > > `tools::addDirectoryList` uses `StringRef::find`, I'm not sure if `StringRef::split` creates real copies of the string...
> > > What is your suggestion?
> > IMO you should use whatever existing code does, in that case `StringRef::find`.
> Is this comment still relevant in the light of the most recent changes?
Probably not (although the code is now completely different from `tools::addDirectoryList`)
Repository:
rC Clang
https://reviews.llvm.org/D43197
More information about the cfe-commits
mailing list