================
@@ -2808,6 +2812,7 @@ void MicrosoftCXXNameMangler::mangleType(const 
BuiltinType *T, Qualifiers,
 
 #define SVE_TYPE(Name, Id, SingletonId) \
   case BuiltinType::Id:
+#define SVE_SCALAR_TYPE(Name, MangledName, Id, SingletonId, Bits)
----------------
MacDue wrote:

With the current approach there simply won't be cases added for other 
`SVE_SCALAR_TYPE` types, so they won't be covered by the switch statement at 
all (which does not appear to have a `default:`). There will probably a warning 
when building clang, but it looks like at runtime the compiler will just 
silently drop mangling these types (and won't error). 

https://github.com/llvm/llvm-project/pull/124968
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to