[PATCH] [Multiprecision Arithmetic Optimization] Teach SelectionDAG how to promote the integer carry result of {add, sub}{c, e} to a larger integer type.

Sean Silva silvas at purdue.edu
Wed May 15 18:33:05 PDT 2013


+  SDValue Ops[3] = { N->getOperand(0), N->getOperand(1) };

Having an array of size 3 with only two initializers seems confusing in the
sense that when I see this I think to myself "is this intentional, or is it
a bug?". Don't make the reader guess whether this is a bug and instead
explicitly list out 3 entries.

-- Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130515/b3aab65a/attachment.html>


More information about the llvm-commits mailing list