[PATCH] D63953: [AMDGPU] LCSSA pass added in preISel.

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jun 28 12:19:39 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/lib/Target/AMDGPU/AMDGPUISelDAGToDAG.cpp:381
 bool AMDGPUDAGToDAGISel::runOnMachineFunction(MachineFunction &MF) {
+#ifndef NDEBUG
+  DominatorTree & DT = getAnalysis<DominatorTreeWrapperPass>().getDomTree();
----------------
arsenm wrote:
> Maybe this should be under EXPENSIVE_CHECKs?
The pass dependency in particular


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D63953/new/

https://reviews.llvm.org/D63953





More information about the llvm-commits mailing list