aaron.ballman added a comment.

Thank you for this coverage as well! Are there issues filed for the FIXME 
comments so we have a more prominent record of the problems?

LGTM modulo comment concerns.



================
Comment at: clang/test/SemaCXX/vector.cpp:554
+void test_int_vector_scalar(unsigned int ua, v2u v2ua) {
+  // Integer vector vs integer operators. These will splat
+  (void)(v2ua + ua);
----------------
Same suggestion here as from the C patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151059

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

Reply via email to