[PATCH] D84322: [AST][RecoveryExpr] Suppress spurious "typecheck_cond_expect_scalar" diagnostic
    Haojian Wu via Phabricator via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Jul 22 06:55:04 PDT 2020
    
    
  
hokein marked an inline comment as done.
hokein added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:8076
+      (Cond.get()->isTypeDependent() || LHS.get()->isTypeDependent() ||
+       RHS.get()->isTypeDependent())) {
+    assert(!getLangOpts().CPlusPlus);
----------------
This follows the same way of how C++ codepath handles dependent code, see the above  Line 8069 [CXXCheckConditionalOperands](https://github.com/llvm/llvm-project/blob/master/clang/lib/Sema/SemaExprCXX.cpp#L6028-L6047).
Repository:
  rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D84322/new/
https://reviews.llvm.org/D84322
    
    
More information about the cfe-commits
mailing list