efriedma added inline comments.
================
Comment at: lib/CodeGen/CGBuiltin.cpp:8182
switch (BuiltinID) {
default: return nullptr;
case NEON::BI__builtin_neon_vbsl_v:
----------------
I'm a little concerned about the overall code structure here; even if moving
the code for the MSVC builtins is enough to fix those builtins specifically, is
it actually impossible to hit this "default"? If it is, can we convert it to
an "unreachable"?
Repository:
rC Clang
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D65403/new/
https://reviews.llvm.org/D65403
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
