[PATCH] D97080: [flang][driver] Add -fintrinsic-modules-path option
Andrzej Warzynski via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Fri Mar 19 10:11:48 PDT 2021
awarzynski added inline comments.
================
Comment at: flang/lib/Frontend/CompilerInvocation.cpp:298
+ driverPath.append("/../include/flang/");
+ return driverPath.str().str();
+}
----------------
Given this [[ https://github.com/llvm/llvm-project/blob/987ee6e3cc1fb672b3ed201e72a5281c2ec88c99/llvm/include/llvm/ADT/SmallString.h#L271-L273 | conversion operator ]], wouldn't `return std::string(driverPath);` be more efficient?
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97080/new/
https://reviews.llvm.org/D97080
More information about the cfe-commits
mailing list