[aarch64] Unguard GEPOpt from O3

lanza · web-flow · commit da385e825186 · 2024-03-25T18:08:36.000-04:00
This chunk of code currently runs only if the optimization mode is O3 AND the EnableGEPOpt flag is set. Given that this is the only use case for the EnableGEPOpt flag, the guarding against O3 is kinda pointless. IF the user wants to enable it then the flag should be sufficient. Reviewers: TNorthover, aeubanks Reviewed By: aeubanks Pull Request: #86588
diff --git a/llvm/lib/Target/AArch64/AArch64TargetMachine.cpp b/llvm/lib/Target/AArch64/AArch64TargetMachine.cpp
@@ -603,7 +603,7 @@ void AArch64PassConfig::addIRPasses() {
       addPass(createFalkorMarkStridedAccessesPass());
   }
 
-  if (TM->getOptLevel() == CodeGenOptLevel::Aggressive && EnableGEPOpt) {
+  if (EnableGEPOpt) {
     // Call SeparateConstOffsetFromGEP pass to extract constants within indices
     // and lower a GEP with multiple indices to either arithmetic operations or
     // multiple GEPs with single index.

Original file line number	Diff line number	Diff line change
`@@ -603,7 +603,7 @@ void AArch64PassConfig::addIRPasses() {`
`603`	`603`	`addPass(createFalkorMarkStridedAccessesPass());`
`604`	`604`	`}`
`605`	`605`
`606`		`- if (TM->getOptLevel() == CodeGenOptLevel::Aggressive && EnableGEPOpt) {`
	`606`	`+ if (EnableGEPOpt) {`
`607`	`607`	`// Call SeparateConstOffsetFromGEP pass to extract constants within indices`
`608`	`608`	`// and lower a GEP with multiple indices to either arithmetic operations or`
`609`	`609`	`// multiple GEPs with single index.`