Skip to content

Fix for Attempt to fix [CGData][MachineOutliner] Global Outlining (#90074) #108037 #108047

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 10, 2024

Conversation

kyulee-com
Copy link
Contributor

@kyulee-com kyulee-com commented Sep 10, 2024

The previous attempt to fix [CGData][MachineOutliner] Global Outlining (#90074) #108037 was incomplete because the ImmutableModuleSummaryIndexWrapperPass is now optional for the MachineOutliner pass.

With this fix, the test file CodeGen/AArch64/O3-pipeline.ll shows no changes compared to its state before [CGData][MachineOutliner] Global Outlining (#90074).

@kyulee-com kyulee-com requested review from ellishg and nico September 10, 2024 15:42
@kyulee-com kyulee-com marked this pull request as ready for review September 10, 2024 15:44
@llvmbot
Copy link
Member

llvmbot commented Sep 10, 2024

@llvm/pr-subscribers-backend-aarch64

Author: Kyungwoo Lee (kyulee-com)

Changes

…0074) #108037


Full diff: https://github.com/llvm/llvm-project/pull/108047.diff

1 Files Affected:

  • (modified) llvm/test/CodeGen/AArch64/O3-pipeline.ll (-1)
diff --git a/llvm/test/CodeGen/AArch64/O3-pipeline.ll b/llvm/test/CodeGen/AArch64/O3-pipeline.ll
index 80e08ee713710c..fb94c040ae341a 100644
--- a/llvm/test/CodeGen/AArch64/O3-pipeline.ll
+++ b/llvm/test/CodeGen/AArch64/O3-pipeline.ll
@@ -16,7 +16,6 @@
 ; CHECK-NEXT: Machine Branch Probability Analysis
 ; CHECK-NEXT: Default Regalloc Eviction Advisor
 ; CHECK-NEXT: Default Regalloc Priority Advisor
-; CHECK-NEXT: Module summary info
 ; CHECK-NEXT:   ModulePass Manager
 ; CHECK-NEXT:     Pre-ISel Intrinsic Lowering
 ; CHECK-NEXT:     FunctionPass Manager

@kyulee-com kyulee-com changed the title Fix for Attempt to fix [CGData][MachineOutliner] Global Outlining (#9… Fix for Attempt to fix [CGData][MachineOutliner] Global Outlining (#90074) #108037 Sep 10, 2024
@alx32
Copy link
Contributor

alx32 commented Sep 10, 2024

LGTM

@kyulee-com kyulee-com merged commit ba2aa1d into llvm:main Sep 10, 2024
9 of 11 checks passed
@kyulee-com kyulee-com deleted the attempt2 branch September 10, 2024 16:11
kyulee-com added a commit that referenced this pull request Sep 10, 2024
The previous `Fix for Attempt to fix [CGData][MachineOutliner] Global
Outlining (#90074) #108037 (#108047)` somehow dropped this file.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants