PR17326, PATCH

Sean Silva silvas at purdue.edu
Sat Dec 14 16:06:54 PST 2013


On Sat, Dec 14, 2013 at 3:02 PM, Nick Lewycky <nicholas at mxc.ca> wrote:

>
> +    }
> +    else
> +      Args.push_back(createCast(Builder, (Value*)AI,
> FFTy->getParamType(i)));
>
> Style issue: if you have braces on one side of an if/else, use braces on
> both sides.


And the `else` should be on the same line as the `if`'s brace. Obligatory
suggestion to run clang-format.

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


More information about the llvm-commits mailing list