sidneym added inline comments.
Comment at: clang/lib/Driver/ToolChains/Hexagon.cpp:604
+ const bool HasSysRoot = !D.SysRoot.empty();
+ if (HasSysRoot) {
SmallString<128> P(D.SysRoot);
bcain wrote:
> MaskRay wrote:
> > To match Linux.cpp, this code block
bcain added inline comments.
Comment at: clang/lib/Driver/ToolChains/Hexagon.cpp:604
+ const bool HasSysRoot = !D.SysRoot.empty();
+ if (HasSysRoot) {
SmallString<128> P(D.SysRoot);
MaskRay wrote:
> To match Linux.cpp, this code block should be
MaskRay requested changes to this revision.
MaskRay added a comment.
This revision now requires changes to proceed.
Thanks for the patch. This makes the behavior closer to Linux.cpp . Request
changes to clear my queue before a test is added...
Comment at:
bcain added a comment.
This patch is so far just for discussion, still needs tests, etc. The
motivation for this patch is to be able to include the intrinsic header files
installed in the resource dir.
I've taken inspiration from `ToolChains/Linux.cpp` here.
Repository:
rG LLVM Github
bcain created this revision.
bcain added reviewers: sidneym, MaskRay.
bcain requested review of this revision.
Herald added a project: clang.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D106737
Files:
clang/lib/Driver/ToolChains/Hexagon.cpp
Index:
