erichkeane added inline comments.
================
Comment at: clang/lib/Sema/SemaExpr.cpp:11148
if (LHSType->isVLSTBuiltinType() && RHSType->isVLSTBuiltinType() &&
+ LHSBuiltinTy && RHSBuiltinTy &&
Context.getBuiltinVectorTypeInfo(LHSBuiltinTy).EC !=
----------------
I think this is unnecessary. `isVLSTBuiltinType` only returns true if
`LHSType` is a `BuiltinType` already (or at least, a subset-of). See :
https://clang.llvm.org/doxygen/Type_8cpp_source.html#l02409
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D152197/new/
https://reviews.llvm.org/D152197
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
