[llvm] r257278 - Fix a control flow problem in commit rL257277.

Chen Li via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 13 09:52:51 PST 2016


> On Jan 11, 2016, at 11:26 AM, Chen Li <meloli87 at gmail.com> wrote:
> 
>> 
>> On Jan 10, 2016, at 9:04 PM, David Blaikie <dblaikie at gmail.com <mailto:dblaikie at gmail.com>> wrote:
>> 
>> 
>> 
>> On Sat, Jan 9, 2016 at 10:13 PM, Chen Li via llvm-commits <llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>> wrote:
>> Author: chenli
>> Date: Sun Jan 10 00:13:32 2016
>> New Revision: 257278
>> 
>> URL: http://llvm.org/viewvc/llvm-project?rev=257278&view=rev <http://llvm.org/viewvc/llvm-project?rev=257278&view=rev>
>> Log:
>> Fix a control flow problem in commit rL257277.
>> 
>> 
>> Modified:
>>     llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp
>> 
>> Modified: llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp
>> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp?rev=257278&r1=257277&r2=257278&view=diff <http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp?rev=257278&r1=257277&r2=257278&view=diff>
>> ==============================================================================
>> --- llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp (original)
>> +++ llvm/trunk/lib/Transforms/Utils/SimplifyCFG.cpp Sun Jan 10 00:13:32 2016
>> @@ -3247,6 +3247,8 @@ bool SimplifyCFGOpt::SimplifyResume(Resu
>>             RI->getValue() == RI->getParent()->getFirstNonPHI())
>>      // The resume must unwind the exception that caused control to branch here.
>>      return SimplifySingleResume(RI);
>> +  else
>> +    return false;
>> 
>> Drop the else-after-return, and simply "return false" unconditionally outside the if.
> 
> Fixed on rL257366. Thanks!

CC llvm-commits.

> 
> chen
> 
>>  
>>  }
>> 
>>  // Simplify resume that is shared by several landing pads (phi of landing pad).
>> 
>> 
>> _______________________________________________
>> llvm-commits mailing list
>> llvm-commits at lists.llvm.org <mailto:llvm-commits at lists.llvm.org>
>> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits <http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160113/02ffdcd0/attachment.html>


More information about the llvm-commits mailing list