[PATCH] Fix use of DemandedMask to NewMask for SIGN_EXTEND_INREG

Nadav Rotem nrotem at apple.com
Thu Jan 8 16:01:11 PST 2015


> On Jan 8, 2015, at 1:20 AM, Michael Kuperstein <michael.m.kuperstein at intel.com> wrote:
> 
> We've tried to modify the test-cases, but we can't actually get it to fail.
> We could try to track down why (I guess some earlier transformation changes the code for the problematic cases, where there are multiple uses, into something nicer) but I'm not sure it's worth the effort, since, as you say, this is an obvious bug.
> 
> Would you be ok with committing it without a test?

Okay. 

> 
> 
> http://reviews.llvm.org/D6321
> 
> EMAIL PREFERENCES
>  http://reviews.llvm.org/settings/panel/emailpreferences/
> 
> 





More information about the llvm-commits mailing list