[PATCH] D46775: [LICM] Preserve DT and LoopInfo specifically

Jun Bum Lim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed May 16 14:23:05 PDT 2018


junbuml added inline comments.


================
Comment at: test/Transforms/LICM/pr37323.ll:1-2
+;RUN: opt -loop-simplify -postdomtree -licm -adce -S -o - %s | FileCheck %s
+;RUN: opt -passes='loop-simplify,require<postdomtree>,require<opt-remark-emit>,loop(licm),function(adce)' -S -o - %s | FileCheck %s
+
----------------
mzolotukhin wrote:
> By the way, do we have something like `-verify-dom-tree` for PostDT? It would be cleaner to explicitly verify postdom tree here than calling ADCE and hoping it crashes.
The same tag (-verify-dom-info) seem to cover PostDT's verifyAnalysis as well.  Using -verify-dom-info, I confirmed that this test crash in PostDT's verifyAnalysis without this change.


https://reviews.llvm.org/D46775





More information about the llvm-commits mailing list