aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorLei Wang <wlei@fb.com>2024-04-09 18:53:09 -0700
committerGitHub <noreply@github.com>2024-04-09 18:53:09 -0700
commit892f01a7437b20f5df3edf53824a53889f733b06 (patch)
treea914576b4b8d5439aa183b385350c32017cd76b5
parent36e25772ddd049c8c742e55fbd2b3c9aaceb7060 (diff)
downloadllvm-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.h10
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");