[LLVMdev] warning from -instcombine

[Török Edwin]

On 2009-02-27 21:33, Chris Lattner wrote:
> On Feb 27, 2009, at 10:50 AM, Devang Patel wrote:
>
>   
>>>> So should it be removed, or downgraded to DOUT, or something?
>>>>         
>>> We should not remove them. These indicate a bug in the source,
>>> frontend, or earlier optimization passes. It's pretty useful.
>>>       
>> Why not use assert() like mechanism to catch such bugs?
>>     
>
> The compiler should not crash on invalid or poorly formed code.

How about setting an error flag, and error message in the Module, then
let the caller know the Module is broken?
Sort of like what should happen on invalid inline assembly  instead of
abort(), as we already talked about.

Best regards,
--Edwin