[PATCH] D145848: [Driver] Correct -f(no-)xray-function-index behavior

2023-03-11 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy created this revision. ilammy added reviewers: MaskRay, ianlevesque, dberris. ilammy added a project: clang. Herald added a subscriber: arphaman. Herald added a project: All. ilammy requested review of this revision. Herald added a subscriber: cfe-commits. This option has undergone several

[PATCH] D145849: [Driver][xray] Allow XRay on Apple Silicon

2023-03-11 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy created this revision. ilammy added reviewers: MaskRay, ianlevesque, dberris. ilammy added a project: clang. Herald added a subscriber: kristof.beyls. Herald added a project: All. ilammy requested review of this revision. Herald added a subscriber: cfe-commits. Codegen can handle XRay for A

[PATCH] D145848: [Driver] Correct -f(no-)xray-function-index behavior

2023-03-13 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy updated this revision to Diff 504609. ilammy added a comment. Addressed feedback by @MaskRay. Thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145848/new/ https://reviews.llvm.org/D145848 Files: clang/include/clang/Driver/Options.td clang/test/CodeGen/xray-function-inde

[PATCH] D145849: [Driver][xray] Allow XRay on Apple Silicon

2023-06-17 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy added a comment. In D145849#4412286 , @MaskRay wrote: > However, `Triple.isMacOSX()` is allowed before the feature actually works and > `compiler-rt/test/xray/lit.cfg.py` tests it. I have some patches for `compiler-rt` cooking, fixing the build f

[PATCH] D145848: [Driver] Correct -f(no-)xray-function-index behavior

2023-06-11 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy updated this revision to Diff 530284. ilammy added a comment. Addressing feedback by @MaskRay: - Moved `// REQUIRES:` directive to the top of the test file CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145848/new/ https://reviews.llvm.org/D145848 Files: clang/include/clang/Dr

[PATCH] D145849: [Driver][xray] Allow XRay on Apple Silicon

2023-06-11 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy updated this revision to Diff 530285. ilammy added a comment. Addressing feedback by @MaskRay: - Moved `// REQUIRES:` directive to the top of the test file - Replaced legacy `-target` option with proper `--target` in tests - Also rebased on updated trunk CHANGES SINCE LAST ACTION http

[PATCH] D145848: [Driver] Correct -f(no-)xray-function-index behavior

2023-06-15 Thread Oleksii Lozovskyi via Phabricator via cfe-commits
ilammy abandoned this revision. ilammy added a comment. @MaskRay, thanks! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145848/new/ https://reviews.llvm.org/D145848 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.or