[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-04 Thread Ellis Hoag via Phabricator via cfe-commits
ellis added a comment. In D130808#3700074 , @thakis wrote: > Reverted in 0eb7d86f5873ce897894339a3cc5bc69ca507bee > for now. Thanks for reverting. I'm looking into it. Repository:

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-04 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Reverted in 0eb7d86f5873ce897894339a3cc5bc69ca507bee for now. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130808/new/ https://reviews.llvm.org/D130808 ___ cfe-commits mailing li

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-04 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. Looks like this might break tests on windows: http://45.33.8.238/win/63804/step_7.txt Please take a look and revert for now if it takes a while to fix. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130808/new/ https://revi

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-04 Thread Ellis Hoag 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 rGb692312ca432: [InstrProf] Add new format for -fprofile-list= (authored by ellis). Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-04 Thread Ellis Hoag via Phabricator via cfe-commits
ellis updated this revision to Diff 449994. ellis added a comment. Fix case. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130808/new/ https://reviews.llvm.org/D130808 Files: clang/docs/UsersManual.rst clang/include/clang/Basic/ProfileList.h

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-03 Thread Petr Hosek via Phabricator via cfe-commits
phosek accepted this revision. phosek added a comment. This revision is now accepted and ready to land. LGTM Comment at: clang/include/clang/Basic/ProfileList.h:31-38 + enum ExclusionType { +/// Profiling is allowed. +ALLOW, +/// Profiling is skipped using the \p s

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-08-02 Thread Ellis Hoag via Phabricator via cfe-commits
ellis updated this revision to Diff 449469. ellis added a comment. Rebase and rename `omitprofile` => `skipprofile`. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D130808/new/ https://reviews.llvm.org/D130808 Files: clang/docs/UsersManual.rst c

[PATCH] D130808: [InstrProf] Add new format for -fprofile-list=

2022-07-29 Thread Ellis Hoag via Phabricator via cfe-commits
ellis created this revision. Herald added a project: All. ellis edited the summary of this revision. ellis added a reviewer: phosek. ellis edited the summary of this revision. ellis added a reviewer: davidxl. ellis published this revision for review. Herald added a project: clang. Herald added a su