[clang] [clang] Remove isOSWindows() checks (PR #129909)

2025-04-14 Thread Prabhu Rajasekaran via cfe-commits
Prabhuk wrote: @rnk @mstorsjo -- Can you please take a look at the changes? https://github.com/llvm/llvm-project/pull/129909 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] Remove isOSWindows() checks (PR #129909)

2025-03-07 Thread Martin Storsjö via cfe-commits
@@ -4769,31 +4769,32 @@ ExprResult Sema::CheckOSLogFormatStringArg(Expr *Arg) { /// Check that the user is calling the appropriate va_start builtin for the /// target and calling convention. static bool checkVAStartABI(Sema &S, unsigned BuiltinID, Expr *Fn) { - const llvm::Tri

[clang] [clang] Remove isOSWindows() checks (PR #129909)

2025-03-05 Thread Reid Kleckner via cfe-commits
@@ -4769,31 +4769,32 @@ ExprResult Sema::CheckOSLogFormatStringArg(Expr *Arg) { /// Check that the user is calling the appropriate va_start builtin for the /// target and calling convention. static bool checkVAStartABI(Sema &S, unsigned BuiltinID, Expr *Fn) { - const llvm::Tri

[clang] [clang] Remove isOSWindows() checks (PR #129909)

2025-03-05 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Prabhuk (Prabhuk) Changes The logic to decide which va_start builtin should be used must rely on the ABI information. The target triple based check is rigid. It makes the support for new triples such as x86_64-uefi to add more triple based

[clang] [clang] Remove isOSWindows() checks (PR #129909)

2025-03-05 Thread via cfe-commits
https://github.com/Prabhuk created https://github.com/llvm/llvm-project/pull/129909 The logic to decide which va_start builtin should be used must rely on the ABI information. The target triple based check is rigid. It makes the support for new triples such as x86_64-uefi to add more triple base