[PATCH] D106737: [clang] [hexagon] Add resource include dir

2021-08-10 Thread Brian Cain via Phabricator via cfe-commits
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

[PATCH] D106737: [clang] [hexagon] Add resource include dir

2021-08-09 Thread Fangrui Song via Phabricator via cfe-commits
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:[^"]+]]" +//

[PATCH] D106737: [clang] [hexagon] Add resource include dir

2021-08-09 Thread Brian Cain via Phabricator via cfe-commits
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

[PATCH] D106737: [clang] [hexagon] Add resource include dir

2021-08-06 Thread Fangrui Song via Phabricator via cfe-commits
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 ___

[PATCH] D106737: [clang] [hexagon] Add resource include dir

2021-08-05 Thread Brian Cain via Phabricator via cfe-commits
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