[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