[PATCH] D98389: [IndirectCallPromotion] Don't strip ".__uniq." suffix when it strips ".llvm." suffix.

Thu Mar 11 14:58:32 PST 2021

wmi added a comment.

In D98389#2620644 <https://reviews.llvm.org/D98389#2620644>, @mstorsjo wrote:

> FWIW, I'm seeing errors like "Malformed instrumentation profile data" when building code for a `i686-w64-mingw32` target after this commit. I've yet to reduce the actual trigger for it, but I thought I'd let you know...

Thanks Martin for reporting the error. I think the error is thrown in the call of addFuncName(OtherFuncName) here: 
https://github.com/llvm/llvm-project/blob/b552adf8b388a4fbdaa6fb46bdedc83fc738fc2b/llvm/lib/ProfileData/InstrProf.cpp#L376

Could you help to get the "PGOFuncName" string before addFuncName throws the error? Then we can know what is wrong.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D98389/new/

https://reviews.llvm.org/D98389