================
@@ -5193,12 +5193,12 @@ bool Sema::CheckCallingConvAttr(const ParsedAttr 
&Attrs, CallingConv &CC,
     CC = CC_X86RegCall;
     break;
   case ParsedAttr::AT_MSABI:
-    CC = Context.getTargetInfo().getTriple().isOSWindows() ? CC_C :
-                                                             CC_Win64;
+    CC = Context.getTargetInfo().getTriple().isOSWindowsOrUEFI() ? CC_C
----------------
frobtech wrote:

This really highlights how `getTargetInfo().getCallingConv()` is what we ought 
to have.  For now it can be defined just this way, but in future it might be 
controlled by switches within a given target like UEFI.  But right now, it 
makes code like this and `checkVAStartABI` much clearer.

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

Reply via email to