
<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div><div>On Jul 28, 2011, at 2:56 PM, Jakub Staszak wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; ">Is there any reason that we don't have isLoopSimplifyForm method in LoopBase? Can I move it there?<br><br><div>

<span class="Apple-style-span" style="border-collapse: separate; font-family: Helvetica; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: 2; text-align: -webkit-auto; text-indent: 0px; text-transform: none; white-space: normal; widows: 2; word-spacing: 0px; -webkit-border-horizontal-spacing: 0px; -webkit-border-vertical-spacing: 0px; -webkit-text-decorations-in-effect: none; -webkit-text-size-adjust: auto; -webkit-text-stroke-width: 0px; font-size: medium; "><div>-Kuba</div></span>

</div>

</div></blockquote></div><br><div>I'll take a wild guess that it's because LoopSimplify doesn't run on MachineLoops and lowering passes can invalidate it. You can always test for individual loop properties that you need instead of calling isLoopSimplify. For example, do you *really* need hasDedicatedExits()? If your MachineLoops don't need it, then you should leave it's implementation LoopInfo.cpp.</div><div><br></div><div>-Andy</div></body></html>