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