diff options
| author | wlei <wlei@fb.com> | 2021-11-08 23:05:16 -0800 |
|---|---|---|
| committer | wlei <wlei@fb.com> | 2021-11-12 12:18:43 -0800 |
| commit | aab1810006a6788e32ee04e7d40d0b2474754aa2 (patch) | |
| tree | 04563c08aeee71cb8f74ab5680580da0f6b15f2b /llvm/lib/Transforms/Utils/LoopUnrollRuntime.cpp | |
| parent | 59b1e98137e961a61ffaeb609b6790999c461bec (diff) | |
| download | llvm-aab1810006a6788e32ee04e7d40d0b2474754aa2.zip llvm-aab1810006a6788e32ee04e7d40d0b2474754aa2.tar.gz llvm-aab1810006a6788e32ee04e7d40d0b2474754aa2.tar.bz2 | |
[llvm-profgen] Fix bug of setting function entry
Previously we set `isFuncEntry` flag to true when the funcName from DWARF is equal to the name in symbol table and we use this flag to ignore reporting callsite sample that's from an intra func branch. However, in HHVM, it appears that the symbol table name is inconsistent with the dwarf info func name, it's likely due to `OptimizeGlobalAliases`.
This change is a workaround in llvm-profgen side to mark the only one range as the function entry and add warnings for the remaining inconsistence.
This also fixed a missing `getCanonicalFnName` for symbol name which caused the mismatching as well.
Reviewed By: hoy, wenlei
Differential Revision: https://reviews.llvm.org/D113492
Diffstat (limited to 'llvm/lib/Transforms/Utils/LoopUnrollRuntime.cpp')
0 files changed, 0 insertions, 0 deletions