diff options
| author | Ellis Hoag <ellis.sparky.hoag@gmail.com> | 2022-07-29 16:23:46 -0700 |
|---|---|---|
| committer | Ellis Hoag <ellis.sparky.hoag@gmail.com> | 2022-08-04 08:45:27 -0700 |
| commit | 12e78ff88105f2dc6cb1449d6fcd5d8f69e0512f (patch) | |
| tree | 652e87fc7305b7d2d10eb24ca56bffe00af4fb11 /llvm/lib/Bitcode/Reader/BitcodeReader.cpp | |
| parent | ec7f4a7c5d9794c9fdf4f894873e7edbbfddf3e2 (diff) | |
| download | llvm-12e78ff88105f2dc6cb1449d6fcd5d8f69e0512f.zip llvm-12e78ff88105f2dc6cb1449d6fcd5d8f69e0512f.tar.gz llvm-12e78ff88105f2dc6cb1449d6fcd5d8f69e0512f.tar.bz2 | |
[InstrProf] Add the skipprofile attribute
As discussed in [0], this diff adds the `skipprofile` attribute to
prevent the function from being profiled while allowing profiled
functions to be inlined into it. The `noprofile` attribute remains
unchanged.
The `noprofile` attribute is used for functions where it is
dangerous to add instrumentation to while the `skipprofile` attribute is
used to reduce code size or performance overhead.
[0] https://discourse.llvm.org/t/why-does-the-noprofile-attribute-restrict-inlining/64108
Reviewed By: phosek
Differential Revision: https://reviews.llvm.org/D130807
Diffstat (limited to 'llvm/lib/Bitcode/Reader/BitcodeReader.cpp')
| -rw-r--r-- | llvm/lib/Bitcode/Reader/BitcodeReader.cpp | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp index 343d49b..379614b 100644 --- a/llvm/lib/Bitcode/Reader/BitcodeReader.cpp +++ b/llvm/lib/Bitcode/Reader/BitcodeReader.cpp @@ -1919,6 +1919,8 @@ static Attribute::AttrKind getAttrFromCode(uint64_t Code) { return Attribute::NoCfCheck; case bitc::ATTR_KIND_NO_PROFILE: return Attribute::NoProfile; + case bitc::ATTR_KIND_SKIP_PROFILE: + return Attribute::SkipProfile; case bitc::ATTR_KIND_NO_UNWIND: return Attribute::NoUnwind; case bitc::ATTR_KIND_NO_SANITIZE_BOUNDS: |