[PATCH] D110692: [AMDGPU] Enable machine verification after AMDGPUISelDAGToDAG
Jay Foad via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Sep 29 03:45:01 PDT 2021
foad created this revision.
foad added reviewers: arsenm, rampitec.
Herald added subscribers: kerbowa, hiraditya, t-tye, tpr, dstuttard, yaxunl, nhaehnle, jvesely, kzhuravl.
foad requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.
This was introduced in D32628 <https://reviews.llvm.org/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.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D110692
Files:
llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
Index: llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
===================================================================
--- llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
+++ llvm/lib/Target/AMDGPU/AMDGPUTargetMachine.cpp
@@ -1020,8 +1020,7 @@
}
bool AMDGPUPassConfig::addInstSelector() {
- // Defer the verifier until FinalizeISel.
- addPass(createAMDGPUISelDag(&getAMDGPUTargetMachine(), getOptLevel()), false);
+ addPass(createAMDGPUISelDag(&getAMDGPUTargetMachine(), getOptLevel()));
return false;
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D110692.375825.patch
Type: text/x-patch
Size: 519 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210929/a1da8a07/attachment-0001.bin>
More information about the llvm-commits
mailing list