[llvm-commits] [PATCH] Bug fix in ARMISelDAGToDAG.cpp

Jim Grosbach grosbach at apple.com
Thu May 6 08:34:17 PDT 2010


Thanks! Great catch. Applied in r103181.


On May 6, 2010, at 4:14 AM, Edmund Grimley-Evans wrote:

> The "break" in line 1869 used to fall through to the "return
> SelectCode(N)" in line 1998, but then someone inserted the "case
> ISD::CONCAT_VECTORS" so that it falls through to the "return
> SelectConcatVector(N)" in line 1994 instead and much hilarity ensues.
> 
> This is one way of fixing it. Or you might prefer to insert a "break"
> at line 1991, or even use a "goto".
> 
> 
> 
> 
> -- 
> IMPORTANT NOTICE: The contents of this email and any attachments are confidential and may also be privileged. If you are not the intended recipient, please notify the sender immediately and do not disclose the contents to any other person, use it for any purpose, or store or copy the information in any medium.  Thank you.
> <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