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

Bob Wilson bob.wilson at apple.com
Thu May 6 08:31:46 PDT 2010


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

Thanks.  Of those options, I prefer inserting a break.  I'll put that in.



More information about the llvm-commits mailing list