[PATCH] D84226: [AST][RecoveryExpr] Support dependent binary operator in C for error recovery.

Haojian Wu via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jul 21 02:44:43 PDT 2020


hokein marked an inline comment as done.
hokein added inline comments.


================
Comment at: clang/lib/Sema/SemaExpr.cpp:14243
+  // FIXME: this early typo correction can be removed now, since we support the
+  // dependent binary operator in C.
   std::tie(LHS, RHS) = CorrectDelayedTyposInBinOp(*this, Opc, LHSExpr, RHSExpr);
----------------
I think we probably need a new cc1 flag e.g. `CDependenceType` to control the typo-correction and whether we build dependent ast node for C.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84226





More information about the cfe-commits mailing list