[PATCH] D148188: [llvm-profdata] Make profile reader behavior consistent when encountering malformed profiles

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 25 17:16:18 PDT 2023


wenlei added inline comments.
Herald added a subscriber: hoy.


================
Comment at: llvm/lib/ProfileData/SampleProfReader.cpp:743-744
-    bool UseMD5 = hasSecFlag(Entry, SecNameTableFlags::SecFlagMD5Name);
-    assert((!FixedLengthMD5 || UseMD5) &&
-           "If FixedLengthMD5 is true, UseMD5 has to be true");
     FunctionSamples::HasUniqSuffix =
----------------
Why is assertion removed? Is it no longer the case? 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148188



More information about the llvm-commits mailing list