[PATCH] D131195: [InstrProf][attempt 2] Add new format for -fprofile-list=
Ellis Hoag via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 4 12:51:19 PDT 2022
ellis created this revision.
Herald added a project: All.
ellis added reviewers: phosek, davidxl, thakis.
ellis published this revision for review.
ellis added inline comments.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Comment at: clang/test/CodeGen/profile-filter-new.c:14
+// RUN: echo "[llvm]" > %t2.list
+// RUN: echo "source:%s=forbid" | sed -e 's/\\/\\\\/g' >> %t2.list
+// RUN: echo "function:foo=allow" >> %t2.list
I think the Windows test from D130808 broke because this `sed` command was interacting badly with newlines. Using multiple `echo` commands seems to work.
In D130807 <https://reviews.llvm.org/D130807> we added the `skipprofile` attribute. This commit
changes the format so we can either `forbid` or `skip` profiling
functions by adding the `noprofile` or `skipprofile` attributes,
respectively. The behavior of the original format remains
Also, add the `skipprofile` attribute when using
This was originally landed as https://reviews.llvm.org/D130808 but was
reverted due to a Windows test failure.
rG LLVM Github Monorepo
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 15934 bytes
Desc: not available
More information about the cfe-commits