[PATCH] D50767: [C++2a] Parsing and semantic analysis for contracts (P0542R5)

[Stephen Kelly via Phabricator via cfe-commits]

steveire added inline comments.


================
Comment at: lib/Parse/ParseStmt.cpp:272
     bool HasLeadingEmptyMacro = Tok.hasLeadingEmptyMacro();
-    return Actions.ActOnNullStmt(ConsumeToken(), HasLeadingEmptyMacro);
+    SourceLocation SemiLoc = ConsumeToken();
+    return Actions.ActOnNullStmt(SemiLoc, HasLeadingEmptyMacro,
----------------
Any reason to create the intermediate variable here, instead of leaving the `ConsumeToken()` call where it was?


Repository:
  rC Clang

https://reviews.llvm.org/D50767