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

Reply via email to