belickim added inline comments.
================
Comment at: include/clang/Basic/TargetBuiltins.h:154
+ /// \brief Nios2 builtins
+ namespace Nios2 {
+ enum {
----------------
ahatanak wrote:
> No need to indent enum inside namespace.
>
> http://llvm.org/docs/CodingStandards.html#namespace-indentation
I was trying to follow conventions used by other code in this file (this also
applies to some extend to other things that you pointed out). I will try to
follow LLVM coding conventions over style present in file for any future
changes. Thank you for your comments, I will prepare update with all changes
that you suggested.
https://reviews.llvm.org/D33356
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
