RKSimon added a reviewer: RKSimon. RKSimon added a comment. Tests?
================ Comment at: lib/Sema/SemaStmtAsm.cpp:71 + if (!Piece.isOperand()) + continue; ---------------- This all looks like a clang-format NFC change - just commit it? ================ Comment at: lib/Sema/SemaStmtAsm.cpp:617 + return; + } else if (Res->isRValue()) { + bool Enum = isa<clang::EnumType>(T) && Res->EvaluateAsRValue(Eval, Context); ---------------- (style) Split these instead of an if-elseif chain Repository: rL LLVM https://reviews.llvm.org/D37413 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits