[llvm] r276449 - [PM] Port BreakCriticalEdges to the new PM.

Wei Mi via llvm-commits llvm-commits at lists.llvm.org
Sat Jul 23 09:33:38 PDT 2016


>> --- llvm/trunk/test/Analysis/Dominators/2007-01-14-BreakCritEdges.ll
>> (original)
>> +++ llvm/trunk/test/Analysis/Dominators/2007-01-14-BreakCritEdges.ll Fri
>> Jul 22 13:04:25 2016
>> @@ -1,4 +1,5 @@
>>  ; RUN: opt < %s -domtree -break-crit-edges -domtree -disable-output
>> +; RUN: opt < %s
>> -passes='require<domtree>,break-crit-edges,require<domtree>' -disable-output
>
>
> I don't think we need `require<domtree>` at the end of this pipeline.
>
> -- Sean Silva
>

Right, thanks. -break-crit-edges preserves domtree, so
require<domtree> at the end of pipeline is in vain. Actually the
-domtree at the end of the old pipeline is also in vain. I will remove
both of them.

Wei.


More information about the llvm-commits mailing list