[PATCH] D58827: [Sema][NFCI] Don't allocate storage for the various CorrectionCandidateCallback unless we are going to do some typo correction

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 18 13:58:04 PDT 2019


rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.

I think this is worth the complexity of the repeated clone methods. lgtm



================
Comment at: lib/Sema/SemaType.cpp:5911
       ExprResult AddrSpace = S.ActOnIdExpression(
-          S.getCurScope(), SS, TemplateKWLoc, id, false, false);
+          S.getCurScope(), SS, TemplateKWLoc, id, /*HasTrailingLParen=*/false,
+          /*IsAddressOfOperand=*/false);
----------------
These changes are unrelated, but good.


Repository:
  rC Clang

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D58827/new/

https://reviews.llvm.org/D58827





More information about the cfe-commits mailing list