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

[Tanya Lattner]

On Apr 7, 2011, at 12:34 AM, Cameron Zwarich wrote:

> 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?

Thanks! I assume so, but I'm not an expert in this code area. 

-Tanya

> 
> 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
>