[llvm-commits] [PATCH] fix assertion in jump-threading (PR13405)

Richard Osborne richard at xmos.com
Fri Jul 20 03:45:56 PDT 2012


Thanks for taking a look. Committed in r160546:

http://llvm.org/viewvc/llvm-project?view=rev&revision=160546

On 19/07/12 18:45, Chad Rosier wrote:
> Seems reasonable to me.
>
>   Chad
>
> On Jul 19, 2012, at 10:30 AM, Richard Osborne wrote:
>
>> This fixes an assertion in jump threading (see PR13405). The assertion happens because GetBestDestForJumpOnUndef() assumes there is at least 1 successor, which isn't the case if the block ends in an indirect branch with no successors. This patch fixes the problem by bailing out in this case.
>>
>> OK to commit?
>>
>> -- 
>> Richard Osborne | XMOS
>> http://www.xmos.com
>>
>> <jumpthreading.diff>_______________________________________________
>> llvm-commits mailing list
>> llvm-commits at cs.uiuc.edu
>> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


-- 
Richard Osborne | XMOS
http://www.xmos.com




More information about the llvm-commits mailing list