[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-14 Thread Paul Kirth via Phabricator via cfe-commits
paulkirth added a comment. Sorry, that bot is green now as of 437d4e01fe4c057509dff30efd560049ad07bc99 , so it looks like it was just bad timing that led me to suspect your commit, since it touched the test file. I'm not sure

[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-14 Thread Hans Wennborg via Phabricator via cfe-commits
hans added a comment. In D119574#3316216 , @paulkirth wrote: > Hi, > > I think we hit a test failure due to this patch in Fuchsia's Clang canary for > Windows. We've had a lot of breakages on Windows today, so this was hidden > until 0574b5fc >

[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-11 Thread Paul Kirth via Phabricator via cfe-commits
paulkirth added a comment. Hi, I think we hit a test failure due to this patch in Fuchsia's Clang canary for Windows. We've had a lot of breakages on Windows today, so this was hidden until 0574b5fc landed. The failing test

[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-11 Thread Nico Weber via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG6f1147f825d0: [clang] Expose -fprofile-use in clang-cl (authored by thakis). Herald added a project: clang. Repository: rG LLVM Github Monorepo C

[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-11 Thread Hans Wennborg via Phabricator via cfe-commits
hans accepted this revision. hans added a comment. This revision is now accepted and ready to land. lgtm CHANGES SINCE LAST ACTION https://reviews.llvm.org/D119574/new/ https://reviews.llvm.org/D119574 ___ cfe-commits mailing list cfe-commits@lists

[PATCH] D119574: [clang] Expose -fprofile-use in clang-cl

2022-02-11 Thread Nico Weber via Phabricator via cfe-commits
thakis created this revision. thakis added a reviewer: hans. Herald added a subscriber: dang. thakis requested review of this revision. Less typing than `-fprofile-instr-use`, and means the same thing. https://reviews.llvm.org/D119574 Files: clang/include/clang/Driver/Options.td clang/test/