fhahn added inline comments.

================
Comment at: clang/lib/Sema/SemaExpr.cpp:7400
+  } else {
+    return Diag(R.getBegin(),
+                diag::err_invalid_conversion_between_matrix_and_scalar)
----------------
When we get here, `SrcTy` may not necessarily be a scalar I think (see my 
earlier note about adding more invalid cast tests for some additional cases 
that might reach this code)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D99037

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

Reply via email to