[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-08-07 Thread Brian Cain via Phabricator via cfe-commits
This revision was not accepted when it landed; it landed in state "Needs Review". This revision was automatically updated to reflect the committed changes. Closed by commit rG5bc4b34a3aa9: [clang][hexagon] Handle library path arguments earlier (authored by androm3da). Repository: rG LLVM

[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-08-07 Thread Sid Manning via Phabricator via cfe-commits
sidneym added a comment. LGTM Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156771/new/ https://reviews.llvm.org/D156771 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-08-03 Thread Brian Cain via Phabricator via cfe-commits
bcain added a comment. ping? any concerns about this change? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156771/new/ https://reviews.llvm.org/D156771 ___ cfe-commits mailing list

[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-08-01 Thread Brian Cain via Phabricator via cfe-commits
bcain updated this revision to Diff 546314. bcain added a comment. Changed test to use `-Werror -###` instead of `-fdriver-only -v -Werror`. Fixed indentation / `clang-format`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156771/new/

[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-08-01 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. (After D156363 , newer tests can use `-### -Werror` instead of `-fdriver-only -v -Werror` :) ) Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D156771/new/ https://reviews.llvm.org/D156771

[PATCH] D156771: [clang][hexagon] Handle library path arguments earlier

2023-07-31 Thread Brian Cain via Phabricator via cfe-commits
bcain created this revision. bcain added reviewers: kparzysz, sidneym, MaskRay. Herald added a project: All. bcain requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. The removal of the early return in 96832a6bf7e0e7f1e8d634d38c44a1b32d512923