diff options
author | Lei Wang <wlei@fb.com> | 2024-04-09 18:53:09 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-04-09 18:53:09 -0700 |
commit | 892f01a7437b20f5df3edf53824a53889f733b06 (patch) | |
tree | a914576b4b8d5439aa183b385350c32017cd76b5 | |
parent | 36e25772ddd049c8c742e55fbd2b3c9aaceb7060 (diff) | |
download | llvm-892f01a7437b20f5df3edf53824a53889f733b06.zip llvm-892f01a7437b20f5df3edf53824a53889f733b06.tar.gz llvm-892f01a7437b20f5df3edf53824a53889f733b06.tar.bz2 |
Remove the assertion to unblock breakages (#88035)
as titled.
-rw-r--r-- | llvm/include/llvm/Transforms/Utils/SampleProfileLoaderBaseImpl.h | 10 |
1 files changed, 3 insertions, 7 deletions
diff --git a/llvm/include/llvm/Transforms/Utils/SampleProfileLoaderBaseImpl.h b/llvm/include/llvm/Transforms/Utils/SampleProfileLoaderBaseImpl.h index 581d354..39295d6 100644 --- a/llvm/include/llvm/Transforms/Utils/SampleProfileLoaderBaseImpl.h +++ b/llvm/include/llvm/Transforms/Utils/SampleProfileLoaderBaseImpl.h @@ -140,13 +140,9 @@ public: // be different and result in different checksums. So we should use the // state from the new (available_externally) function, which is saved in its // attribute. - assert((LTOPhase != ThinOrFullLTOPhase::ThinLTOPostLink || - IsAvailableExternallyLinkage || !Desc || - profileIsHashMismatched(*Desc, Samples) == - F.hasFnAttribute("profile-checksum-mismatch")) && - "In post-link, profile checksum matching state doesn't match the " - "internal function's 'profile-checksum-mismatch' attribute."); - (void)LTOPhase; + // TODO: If the function's profile only exists as nested inlinee profile in + // a different module, we don't have the attr mismatch state(unknown), we + // need to fix it later. if (IsAvailableExternallyLinkage || !Desc) return !F.hasFnAttribute("profile-checksum-mismatch"); |