[llvm] [BOLT] Always run CheckLargeFunctions in non-relocation mode (PR #80922)

Maksim Panchenko via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 8 14:10:48 PST 2024


================
@@ -3631,6 +3631,7 @@ void RewriteInstance::mapCodeSections(BOLTLinker::SectionMapper MapSection) {
     Function.setImageAddress(FuncSection->getAllocAddress());
     Function.setImageSize(FuncSection->getOutputSize());
     if (Function.getImageSize() > Function.getMaxSize()) {
+      assert(!BC->isX86() && "Unexpected large function.");
----------------
maksfb wrote:

To clarify the question, could you test it on your side with assertion changed to check for CIs? Ideally, the check should be properly modified to accommodate for CIs on supported architectures.

https://github.com/llvm/llvm-project/pull/80922


More information about the llvm-commits mailing list