[PATCH] D122844: [llvm-profgen] Fixing a context attribure update issue due to a non-derministic processing order on different platforms.
Wenlei He via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 31 13:40:18 PDT 2022
wenlei added inline comments.
================
Comment at: llvm/tools/llvm-profgen/ProfileGenerator.cpp:729
+ if (WasLeafInlined)
+ FProfile.getContext().setAttribute(ContextWasInlined);
}
----------------
This fix assumes that the 1st is always going to be called for given context, and it's just a matter of order. Is that actually true?
What is the exact call path for both cases?
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122844/new/
https://reviews.llvm.org/D122844
More information about the llvm-commits
mailing list