[PATCH] D48914: [PGOMemOPSize] Preserve the DominatorTree

Chijun Sima via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 8 04:47:33 PDT 2018


NutshellySima added inline comments.


================
Comment at: test/Transforms/PGOProfile/memop_clone.ll:1
-; RUN: opt < %s -pgo-memop-opt -S | FileCheck %s
 
----------------
dmgreen wrote:
> pgo-memop-opt requires LoopInfo, which requires DomTree, right? So this will test preserving the DT. SGTM.
Yes. I also tried with an invalid DomTree, it indeed asserts as expected.


Repository:
  rL LLVM

https://reviews.llvm.org/D48914





More information about the llvm-commits mailing list