[cfe-commits] r134754 - in /cfe/trunk: lib/CodeGen/TargetInfo.cpp test/CodeGen/x86_64-arguments.c

[Chandler Carruth]

On Fri, Jul 8, 2011 at 3:18 PM, Bruno Cardoso Lopes <bruno.cardoso at gmail.com
> wrote:

> +    } else if (Size == 128 | Size == 256) {


I assume you meant "||" here rather than "|"?

GCC actually catches this:

lib/CodeGen/TargetInfo.cpp:1139: error: suggest parentheses around
comparison in operand of '|' [-Wparentheses]

Although I don't see why parentheses are relevant here. Just looks
suspicious when | is used with bool operands inside an if condition...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110711/d36182f5/attachment.html>