[PATCH] D153261: [AMDGPU] Assert only when -verify-dom-info is invoked.
Pravin Jagtap via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Jun 19 04:02:35 PDT 2023
pravinjagtap created this revision.
Herald added subscribers: foad, kerbowa, hiraditya, tpr, dstuttard, yaxunl, jvesely, kzhuravl, arsenm.
Herald added a project: All.
pravinjagtap requested review of this revision.
Herald added subscribers: llvm-commits, wdng.
Herald added a project: LLVM.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D153261
Files:
llvm/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
llvm/test/CodeGen/AMDGPU/global_atomics_iterative_scan.ll
Index: llvm/test/CodeGen/AMDGPU/global_atomics_iterative_scan.ll
===================================================================
--- llvm/test/CodeGen/AMDGPU/global_atomics_iterative_scan.ll
+++ llvm/test/CodeGen/AMDGPU/global_atomics_iterative_scan.ll
@@ -1,5 +1,5 @@
; NOTE: Assertions have been autogenerated by utils/update_test_checks.py
-; RUN: opt -S -mtriple=amdgcn-- -amdgpu-atomic-optimizer-strategy=Iterative -passes=amdgpu-atomic-optimizer %s | FileCheck -check-prefix=IR %s
+; RUN: opt -S -mtriple=amdgcn-- -amdgpu-atomic-optimizer-strategy=Iterative -passes=amdgpu-atomic-optimizer -verify-dom-info %s | FileCheck -check-prefix=IR %s
define amdgpu_kernel void @uniform_value(ptr addrspace(1) , ptr addrspace(1) %val) #0 {
; IR-LABEL: @uniform_value(
Index: llvm/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
===================================================================
--- llvm/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
+++ llvm/lib/Target/AMDGPU/AMDGPUAtomicOptimizer.cpp
@@ -771,7 +771,8 @@
SplitBlockAndInsertIfThen(Cond, &I, false, nullptr, &DTU, nullptr);
// Control flow is changed here after splitting I's block
- assert(DTU.getDomTree().verify(DominatorTree::VerificationLevel::Full));
+ if (VerifyDomInfo)
+ assert(DTU.getDomTree().verify(DominatorTree::VerificationLevel::Full));
// At this point, we have split the I's block to allow one lane in wavefront
// to update the precomputed reduced value. Also, completed the codegen for
@@ -799,7 +800,8 @@
{{DominatorTree::Insert, EntryBB, ComputeLoop},
{DominatorTree::Insert, ComputeLoop, ComputeEnd},
{DominatorTree::Delete, EntryBB, SingleLaneTerminator->getParent()}});
- assert(DTU.getDomTree().verify(DominatorTree::VerificationLevel::Full));
+ if (VerifyDomInfo)
+ assert(DTU.getDomTree().verify(DominatorTree::VerificationLevel::Full));
Predecessor = ComputeEnd;
} else {
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153261.532595.patch
Type: text/x-patch
Size: 1945 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230619/1d82653a/attachment.bin>
More information about the llvm-commits
mailing list