bcain updated this revision to Diff 365459.
bcain added a comment.
Suggestions from Fangrui: override the resource-dir with an explicit arg,
disable for windows.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106737/new/
https://reviews.llvm.org/D1
MaskRay accepted this revision.
MaskRay added inline comments.
This revision is now accepted and ready to land.
Comment at: clang/test/Driver/hexagon-toolchain-linux.c:106
+// CHECK008: InstalledDir: [[INSTALLED_DIR:.+]]
+// CHECK008: "-resource-dir" "[[RESOURCE:[^"]+]]"
+//
bcain updated this revision to Diff 365352.
bcain added a comment.
Test case updates per suggestion.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106737/new/
https://reviews.llvm.org/D106737
Files:
clang/lib/Driver/ToolChains/Hexagon.cpp
clan
MaskRay added a comment.
Thanks for the test. linux-cross.cpp has some `-SAME: {{^}} ` and
`"[[RESOURCE]]/include"` patterns which you may use
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D106737/new/
https://reviews.llvm.org/D106737
___
bcain updated this revision to Diff 364665.
bcain retitled this revision from "DRAFT - [clang] [hexagon] Add resource
include dir" to "[clang] [hexagon] Add resource include dir".
bcain added a comment.
Herald added a subscriber: ormris.
Add a test case, address failing test.
Repository:
rG L