[llvm-commits] [PATCH] ARM DAG Combine on illegal types patch

Cameron Zwarich zwarich at apple.com
Thu Apr 7 00:34:51 PDT 2011


This looks good to me. I had to do the same thing in some ARM-specific DAGCombiner tricks that I added. I assume it still catches the optimization after the vector type has been expanded, right?

Cameron

On 2011-04-05, at 3:49 PM, Tanya Lattner wrote:

> I've attached a patch that includes a fix for 2 crashes in the ARM DAG Combiner when doing an AND or OR combine on an illegal vector type (vectors of size 3). I've also included a test case.
> 
> Please review and let me know if its ok to commit. 
> 
> Thanks,
> Tanya
> 
> <ARMDagCombineIllegalTypes.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list