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



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to