From e7bc73739a00e471cb96527415e6819cebfc7e80 Mon Sep 17 00:00:00 2001 From: Matt Arsenault Date: Fri, 24 Jun 2022 19:03:57 -0400 Subject: [PATCH] GlobalISel: Make LoadStoreOpt preserve all Avoids dropping CSE info analysis --- llvm/lib/CodeGen/GlobalISel/LoadStoreOpt.cpp | 1 + llvm/test/CodeGen/AArch64/GlobalISel/gisel-commandline-option.ll | 3 +-- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/llvm/lib/CodeGen/GlobalISel/LoadStoreOpt.cpp b/llvm/lib/CodeGen/GlobalISel/LoadStoreOpt.cpp index 0e56c78..d4fbf7d 100644 --- a/llvm/lib/CodeGen/GlobalISel/LoadStoreOpt.cpp +++ b/llvm/lib/CodeGen/GlobalISel/LoadStoreOpt.cpp @@ -73,6 +73,7 @@ void LoadStoreOpt::init(MachineFunction &MF) { void LoadStoreOpt::getAnalysisUsage(AnalysisUsage &AU) const { AU.addRequired(); + AU.setPreservesAll(); getSelectionDAGFallbackAnalysisUsage(AU); MachineFunctionPass::getAnalysisUsage(AU); } diff --git a/llvm/test/CodeGen/AArch64/GlobalISel/gisel-commandline-option.ll b/llvm/test/CodeGen/AArch64/GlobalISel/gisel-commandline-option.ll index 10bdf08..422edaa5 100644 --- a/llvm/test/CodeGen/AArch64/GlobalISel/gisel-commandline-option.ll +++ b/llvm/test/CodeGen/AArch64/GlobalISel/gisel-commandline-option.ll @@ -61,9 +61,8 @@ ; VERIFY-O0-NEXT: AArch64O0PreLegalizerCombiner ; VERIFY-NEXT: Verify generated machine code ; ENABLED-O1-NEXT: Basic Alias Analysis (stateless AA impl) -; ENABLED-O1-NEXT: Function Alias Analysis Results +; ENABLED-O1-NEXT: Function Alias Analysis Results ; ENABLED-O1-NEXT: LoadStoreOpt -; ENABLED-O1-NEXT: Analysis containing CSE Info ; VERIFY-O0-NEXT: Analysis containing CSE Info ; ENABLED-NEXT: Legalizer ; VERIFY-NEXT: Verify generated machine code -- 2.7.4