[llvm] f9b6830 - [AMDGPU] Enable machine verification after AMDGPUISelDAGToDAG

Jay Foad via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 29 10:50:37 PDT 2021


Author: Jay Foad
Date: 2021-09-29T18:47:19+01:00
New Revision: f9b68304a22ef82d827d282dd9014efc2e7ac875

URL: https://github.com/llvm/llvm-project/commit/f9b68304a22ef82d827d282dd9014efc2e7ac875
DIFF: https://github.com/llvm/llvm-project/commit/f9b68304a22ef82d827d282dd9014efc2e7ac875.diff

LOG: [AMDGPU] Enable machine verification after AMDGPUISelDAGToDAG

This was introduced in D32628 but it does not seem to be required any
more. At least it does not show any problems in check-llvm in an
LLVM_ENABLE_EXPENSIVE_CHECKS build.

Differential Revision: https://reviews.llvm.org/D110692

Added: 
    

Modified: 
    llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
index 5498df03c7abe..bafafe84cd17c 100644
--- a/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
+++ b/llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
@@ -1020,8 +1020,7 @@ bool AMDGPUPassConfig::addPreISel() {
 }
 
 bool AMDGPUPassConfig::addInstSelector() {
-  // Defer the verifier until FinalizeISel.
-  addPass(createAMDGPUISelDag(&getAMDGPUTargetMachine(), getOptLevel()), false);
+  addPass(createAMDGPUISelDag(&getAMDGPUTargetMachine(), getOptLevel()));
   return false;
 }
 


        


More information about the llvm-commits mailing list