[PATCH] AArch64: Relax assert about large shift sizes.

Quentin Colombet qcolombet at apple.com
Tue Jan 13 17:35:41 PST 2015


Hi Matthias,

On Jan 12, 2015, at 7:06 PM, Matthias Braun <matze at braunis.de> wrote:

> Abandoning in favor of http://reviews.llvm.org/D6946

I saw that in the new approach you’re fixing the DAG combiner, which is good, but I have one question. Did you try your test case with fast-isel?
I am wondering if we might hit the same assert through that path.

Thanks,
-Quentin

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





More information about the llvm-commits mailing list