<div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap;">Hi Sebastian,</span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap;"><br></span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap;">Thank you so much. </span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap;">Could you revise the new attachment and commit it?</span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><br></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap;">Star Tan</span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-spa!
 ce: pre-wrap; line-height: 1.7;"><br></span></div><div style="line-height:1.7;color:#000000;font-size:14px;font-family:arial"><span style="white-space: pre-wrap; line-height: 1.7;">At 2013-06-11 02:21:52,"Sebastian Pop" <<a href="mailto:spop@codeaurora.org">spop@codeaurora.org</a>> wrote:</span><br><pre>>> +    Loop *L = LI->getLoopFor(entry);
>> +
>> +    if (L){
>
>Please add a space between ){
>
>Note that here you are testing for L not NULL...
>
>> +      if (!L->isLoopSimplifyForm())
>> +        INVALID(SimpleLoop, "Loop not in simplify form is invalid!");
>> +
>> +      for (pred_iterator PI = pred_begin(entry), PE = pred_end(entry); PI != PE;
>> +           ++PI) {
>> +        // Region entering edges come from the same loop but outside the region
>> +        // are not allowed.
>> +        if (L && L->contains(*PI) && !R.contains(*PI))
>
>... and here you are testing whether L is not NULL again: please
>remove this check here, as this is inside a loop.</pre><pre>>
>
>-- 
>Qualcomm Innovation Center, Inc. is a member of Code Aurora Forum,
>hosted by The Linux Foundation
</pre></div></div><br><br><span title="neteasefooter"><span id="netease_mail_footer"><span title="neteasefooter"><span id="netease_mail_footer"><a href="#" target="_blank"></a></span></span>
</span></span>