[clang] [clang] Pass fp128 indirectly and return in xmm0 on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
https://github.com/tgross35 updated https://github.com/llvm/llvm-project/pull/115052 >From 6afdfd07a22260914b45363870a7be54324bd736 Mon Sep 17 00:00:00 2001 From: Trevor Gross Date: Tue, 5 Nov 2024 07:00:35 -0500 Subject: [PATCH 1/2] [clang] Add fp128 ABI tests for MinGW (NFC) Add a test based

[clang] [clang] Pass fp128 indirectly and return in xmm0 on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
tgross35 wrote: I updated this PR to pass indirectly and make the xmm0 return explicit, identical to `i128`. This does not match GCC exactly since GCC's `__float128` returns on the stack, unfortunately meaning cross-implementation is still broken. I don't think that needs block the changes to

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-tidy Author: Marco C. (Marcondiro) Changes This reflects the add_new_check.py changes: isLanguageVersionSupported is now overridden by default by the script The changes were instroduced in https://github.com/llvm/llvm-project/pull/100129 Thanks

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
https://github.com/bd1976bris edited https://github.com/llvm/llvm-project/pull/126654 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [analyzer] Limit Store by region-store-binding-limit (PR #127602)

2025-02-28 Thread Balázs Benics via cfe-commits
balazs-benics-sonarsource wrote: Confirmed crash, https://compiler-explorer.com/z/fzoqP36xq https://github.com/llvm/llvm-project/pull/127602 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[clang] [CUDA][HIP] check dtor in deferred diag (PR #129117)

2025-02-28 Thread Yaxun Liu via cfe-commits
https://github.com/yxsamliu updated https://github.com/llvm/llvm-project/pull/129117 >From 6089ec549a2456b6edf716cecf8f77d1ad0923b0 Mon Sep 17 00:00:00 2001 From: "Yaxun (Sam) Liu" Date: Wed, 26 Feb 2025 11:43:28 -0500 Subject: [PATCH] [CUDA][HIP] check dtor in deferred diag Currently the defe

[clang] 751f2fc - Disable unique-object-duplication warning in templates (#129120)

2025-02-28 Thread via cfe-commits
Author: Devon Loehr Date: 2025-02-28T10:25:55+01:00 New Revision: 751f2fc8d5f465be5634b39adb8256a02f419984 URL: https://github.com/llvm/llvm-project/commit/751f2fc8d5f465be5634b39adb8256a02f419984 DIFF: https://github.com/llvm/llvm-project/commit/751f2fc8d5f465be5634b39adb8256a02f419984.diff L

[clang] [clang] Always pass `fp128` arguments indirectly on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
https://github.com/tgross35 updated https://github.com/llvm/llvm-project/pull/115052 >From 60b07161e8668c2bc3ee5d7a4c470a90a7673178 Mon Sep 17 00:00:00 2001 From: Trevor Gross Date: Tue, 5 Nov 2024 07:00:35 -0500 Subject: [PATCH 1/2] [clang] Add fp128 ABI tests for MinGW (NFC) Duplicate `win64

[clang-tools-extra] [clang-tidy] Contributing.rst update snippet and docs (PR #129209)

2025-02-28 Thread Carlos Galvez via cfe-commits
https://github.com/carlosgalvezp approved this pull request. LGTM, thank you! https://github.com/llvm/llvm-project/pull/129209 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [X86][CodeGen] - Use shift operators for const value shifts, instead of built-ins for SSE emulation of MMX intrinsics. (PR #129197)

2025-02-28 Thread Pawan Nirpal via cfe-commits
https://github.com/pawan-nirpal-031 edited https://github.com/llvm/llvm-project/pull/129197 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
@@ -2142,3 +2190,326 @@ std::vector lto::generateModulesOrdering(ArrayRef R) { }); return ModulesOrdering; } + +namespace { +// For this out-of-process backend no codegen is done when invoked for each +// task. Instead we generate the required information (e.g. the summary

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
@@ -186,6 +187,16 @@ BitcodeCompiler::BitcodeCompiler(Ctx &ctx) : ctx(ctx) { std::string(ctx.arg.thinLTOPrefixReplaceNew), std::string(ctx.arg.thinLTOPrefixReplaceNativeObject), ctx.arg.thinLTOEmitImportsFiles, indexFile.get(), onIndexWrite); + } else i

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
@@ -0,0 +1,60 @@ +Distributed ThinLTO (DTLTO) +=== + +DTLTO allows for the distribution of backend ThinLTO compilations via external +distribution systems, e.g. Incredibuild. There is existing support for +distributing ThinLTO compilations by using separate

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
@@ -535,6 +535,21 @@ void tools::gnutools::Linker::ConstructJob(Compilation &C, const JobAction &JA, D.getLTOMode() == LTOK_Thin); } + // Forward the DTLTO options to the linker. We add these unconditionally, + // rather than in addLTOOptions() as it is

[clang] [lld] [llvm] Integrated Distributed ThinLTO (DTLTO): Initial support (PR #126654)

2025-02-28 Thread via cfe-commits
@@ -2142,3 +2190,327 @@ std::vector lto::generateModulesOrdering(ArrayRef R) { }); return ModulesOrdering; } + +namespace { +// For this out-of-process backend no codegen is done when invoked for each +// task. Instead we generate the required information (e.g. the summary

[clang] [NFC][analyzer] OOB test consolidation III: 'outofbound' tests (PR #128508)

2025-02-28 Thread Endre Fülöp via cfe-commits
https://github.com/gamesh411 approved this pull request. This is a nice cleanup change. I have checked the test cases and the reasoning behind the removals. It is also welcome that these reasons are documented, at least in the PR. @NagyDonat What do you think about making these reasoning snippe

[clang] Fix RegionStore assertion failure after #127602 (PR #129224)

2025-02-28 Thread Balazs Benics via cfe-commits
steakhal wrote: FYI @necto @NagyDonat https://github.com/llvm/llvm-project/pull/129224 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Check for uninitialized use in lambda within CXXOperatorCallExpr (PR #129198)

2025-02-28 Thread via cfe-commits
https://github.com/zhaohuiw42 created https://github.com/llvm/llvm-project/pull/129198 Fix #128058. Track whether a LambdaExpr is an immediate operand of a CXXOperatorCallExpr using a new flag, isInCXXOperatorCall. This enables special handling of capture initializations to detect uninitialize

[clang] Reapply "[analyzer] Handle [[assume(cond)]] as __builtin_assume(cond)" (PR #129234)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-static-analyzer-1 Author: Balazs Benics (steakhal) Changes This is the second attempt to bring initial support for [[assume()]] in the Clang Static Analyzer. The first attempt (#116462) was reverted in 2b9abf0db2d106c7208b4372e662ef5df869e6f1 due

[clang] Reapply "[analyzer] Handle [[assume(cond)]] as __builtin_assume(cond)" (PR #129234)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Balazs Benics (steakhal) Changes This is the second attempt to bring initial support for [[assume()]] in the Clang Static Analyzer. The first attempt (#116462) was reverted in 2b9abf0db2d106c7208b4372e662ef5df869e6f1 due to some weird fai

[clang] Reapply "[analyzer] Handle [[assume(cond)]] as __builtin_assume(cond)" (PR #125348)

2025-02-28 Thread Balazs Benics via cfe-commits
steakhal wrote: Reopening this PR as #129234 as I accidentally closed this permanently. https://github.com/llvm/llvm-project/pull/125348 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
https://github.com/devajithvs created https://github.com/llvm/llvm-project/pull/129235 Clang omits the `ULL` suffix when printing large unsigned 64-bit template arguments. This patch ensures that values with the high bit set are explicitly printed with `ULL`.` Based on the original patch by A

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Devajith (devajithvs) Changes Clang omits the `ULL` suffix when printing large unsigned 64-bit template arguments. This patch ensures that values with the high bit set are explicitly printed with `ULL`.` Based on the original patch by Ax

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
https://github.com/devajithvs updated https://github.com/llvm/llvm-project/pull/129235 >From 514ab7d208a039cd660853d222cf1201927bb4cd Mon Sep 17 00:00:00 2001 From: Devajith Valaparambil Sreeramaswamy Date: Fri, 28 Feb 2025 13:13:16 +0100 Subject: [PATCH] [AST] Fix printing of large 64-bit uns

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
https://github.com/devajithvs edited https://github.com/llvm/llvm-project/pull/129235 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Reapply "[analyzer] Handle [[assume(cond)]] as __builtin_assume(cond)" (PR #129234)

2025-02-28 Thread Balazs Benics via cfe-commits
https://github.com/steakhal created https://github.com/llvm/llvm-project/pull/129234 This is the second attempt to bring initial support for [[assume()]] in the Clang Static Analyzer. The first attempt (#116462) was reverted in 2b9abf0db2d106c7208b4372e662ef5df869e6f1 due to some weird failure

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
devajithvs wrote: This will help in removing one of the downstream patches in ROOT: https://github.com/root-project/root/commit/8dff24d0b687e3b2781e00697d32983a9d4d7b52 cc: @vgvassilev https://github.com/llvm/llvm-project/pull/129235 ___ cfe-commits

[clang] Fix RegionStore assertion failure after #127602 (PR #129224)

2025-02-28 Thread Mikael Holmén via cfe-commits
mikaelholmen wrote: I've verified that this patch solves the problem I saw. Thanks! https://github.com/llvm/llvm-project/pull/129224 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [modules] Add missing test file for b21ee08e57173102b67bc18237b135550 (PR #129221)

2025-02-28 Thread Vassil Vassilev via cfe-commits
https://github.com/vgvassilev approved this pull request. LGTM! https://github.com/llvm/llvm-project/pull/129221 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [AST] Fix printing of large 64-bit unsigned template arguments (PR #129235)

2025-02-28 Thread via cfe-commits
https://github.com/devajithvs updated https://github.com/llvm/llvm-project/pull/129235 >From fecba20439aa92078a213a53cdcde79eb8eee801 Mon Sep 17 00:00:00 2001 From: Devajith Valaparambil Sreeramaswamy Date: Fri, 28 Feb 2025 13:13:16 +0100 Subject: [PATCH] [AST] Fix printing of large 64-bit uns

[clang] [clang-repl] Disable EmulatedTLS on Windows for interpreter executor (PR #127468)

2025-02-28 Thread Martin Storsjö via cfe-commits
@@ -597,6 +597,9 @@ llvm::Error Interpreter::CreateExecutor() { auto JTMB = createJITTargetMachineBuilder(TT); if (!JTMB) return JTMB.takeError(); +#if defined(_WIN32) mstorsjo wrote: I'm not sure what the common practice is for the JIT/interpret

[clang] [modules] Add missing test file for b21ee08e57173102b67bc18237b135550 (PR #129221)

2025-02-28 Thread Haojian Wu via cfe-commits
https://github.com/hokein created https://github.com/llvm/llvm-project/pull/129221 The commit missed a test file. >From 7edbcad61eefd03fbb3cdf4d7759b0e41e348946 Mon Sep 17 00:00:00 2001 From: Haojian Wu Date: Fri, 28 Feb 2025 11:20:33 +0100 Subject: [PATCH] [modules] Add missing test file for

[clang] 26fc3aa - [OpenMP] Missing implicit otherwise clause in metadirective. (#127113)

2025-02-28 Thread via cfe-commits
Author: Zahira Ammarguellat Date: 2025-02-28T08:02:35-05:00 New Revision: 26fc3aa983ab4615dfc32cebf74076c118de2a9d URL: https://github.com/llvm/llvm-project/commit/26fc3aa983ab4615dfc32cebf74076c118de2a9d DIFF: https://github.com/llvm/llvm-project/commit/26fc3aa983ab4615dfc32cebf74076c118de2a9d

[clang] [llvm] [OpenMP] Missing implicit otherwise clause in metadirective. (PR #127113)

2025-02-28 Thread Zahira Ammarguellat via cfe-commits
https://github.com/zahiraam closed https://github.com/llvm/llvm-project/pull/127113 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] WIP: Improve the -Wundefined-func-template diagnostic note for invisible template functions (PR #129031)

2025-02-28 Thread Ilya Biryukov via cfe-commits
@@ -5654,6 +5654,8 @@ def warn_func_template_missing : Warning<"instantiation of function %q0 " InGroup, DefaultIgnore; def note_forward_template_decl : Note< "forward declaration of template entity is here">; +def note_unreachable_template_decl +: Note<"declaration of

[clang] [llvm] [X86][CodeGen] - Use shift operators for const value shifts, instead of built-ins for SSE emulation of MMX intrinsics. (PR #129197)

2025-02-28 Thread Evgenii Kudriashov via cfe-commits
@@ -1115,11 +1115,11 @@ _mm_srl_si64(__m64 __m, __m64 __count) /// \param __count ///A 32-bit integer value. /// \returns A 64-bit integer vector containing the right-shifted value. -static __inline__ __m64 __DEFAULT_FN_ATTRS_SSE2 -_mm_srli_si64(__m64 __m, int __count) -{ -

[clang] [llvm] [X86][CodeGen] - Use shift operators for const value shifts, instead of built-ins for SSE emulation of MMX intrinsics. (PR #129197)

2025-02-28 Thread Evgenii Kudriashov via cfe-commits
https://github.com/e-kud edited https://github.com/llvm/llvm-project/pull/129197 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Offload] Always consider `flto` on for AMDGPU (PR #129118)

2025-02-28 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > We do have some framework teams that are still using non-LTO (or non-gpu-rdc) > build. This doesn't force that, adding `-flto` just informs the compiler to forward certain flags to the linker. https://github.com/llvm/llvm-project/pull/129118 __

[clang] [Clang] Add GCC's __builtin_stack_address() to Clang. (PR #121332)

2025-02-28 Thread via cfe-commits
https://github.com/aalhwc closed https://github.com/llvm/llvm-project/pull/121332 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [CUDA][HIP] check dtor in deferred diag (PR #129117)

2025-02-28 Thread Yaxun Liu via cfe-commits
@@ -1798,6 +1798,62 @@ class DeferredDiagnosticsEmitter Inherited::visitUsedDecl(Loc, D); } + // Visitor member and parent dtors called by this dtor. + void VisitCalledDestructors(CXXDestructorDecl *DD) { +const CXXRecordDecl *RD = DD->getParent(); + +// Visi

[clang] [clang] Pass fp128 indirectly and return in xmm0 on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
https://github.com/tgross35 updated https://github.com/llvm/llvm-project/pull/115052 >From 6afdfd07a22260914b45363870a7be54324bd736 Mon Sep 17 00:00:00 2001 From: Trevor Gross Date: Tue, 5 Nov 2024 07:00:35 -0500 Subject: [PATCH 1/2] [clang] Add fp128 ABI tests for MinGW (NFC) Add a test based

[clang] [Clang] Check for uninitialized use in lambda within CXXOperatorCallExpr (PR #129198)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: zhaohui (zhaohuiw42) Changes Fix #128058. Track whether a LambdaExpr is an immediate operand of a CXXOperatorCallExpr using a new flag, isInCXXOperatorCall. This enables special handling of capture initializations to detect uninitialized

[clang] [llvm] [Coroutines] Mark parameter allocas with coro.outside.frame metadata (PR #127653)

2025-02-28 Thread Chuanqi Xu via cfe-commits
ChuanqiXu9 wrote: > > Since the case a coroutine without a suspend point is relatively rare case > > to me. > > For what it's worth, this issue isn't limited to coroutines without any > suspend point. It can also occur: > > * If the coroutine conditionally doesn't await (e.g., `if (condition)

[clang] [modules] Add missing test file for b21ee08e57173102b67bc18237b135550 (PR #129221)

2025-02-28 Thread Haojian Wu via cfe-commits
https://github.com/hokein closed https://github.com/llvm/llvm-project/pull/129221 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] WIP: Improve the -Wundefined-func-template diagnostic note for invisible template functions (PR #129031)

2025-02-28 Thread Ilya Biryukov via cfe-commits
@@ -5082,11 +5085,16 @@ void Sema::InstantiateFunctionDefinition(SourceLocation PointOfInstantiation, if (AtEndOfTU && !getDiagnostics().hasErrorOccurred() && !getSourceManager().isInSystemHeader(PatternDecl->getBeginLoc())) { Diag(PointOfInstantiation,

[clang] a199791 - [modules] Add missing test file for b21ee08e57173102b67bc18237b135550 (#129221)

2025-02-28 Thread via cfe-commits
Author: Haojian Wu Date: 2025-02-28T13:50:18+01:00 New Revision: a19979166c343822be5cb7744da322d2eddff3bc URL: https://github.com/llvm/llvm-project/commit/a19979166c343822be5cb7744da322d2eddff3bc DIFF: https://github.com/llvm/llvm-project/commit/a19979166c343822be5cb7744da322d2eddff3bc.diff LO

[clang] Improve the -Wundefined-func-template diagnostic note for invisible template functions (PR #129031)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-modules Author: Haojian Wu (hokein) Changes See discussion in https://github.com/llvm/llvm-project/issues/125071. Makes the note clear for unreachable case: Before: ``` ./hoge.h:5:12: warning: instantiation of function 'x' required here, bu

[clang] [llvm] [Coroutines] Replace coro.outside.frame metadata with an intrinsic (PR #129255)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-coroutines Author: Hans Wennborg (zmodem) Changes Metadata should not be "load bearing", i.e. required for correctness, since optimizations are not required to preserve it. Therefore, turn this into an intrinsic instead. This is a follow-up to #127653

[clang] dea08c2 - Fix RegionStore assertion failure after #127602 (#129224)

2025-02-28 Thread via cfe-commits
Author: Balázs Benics Date: 2025-02-28T15:48:31+01:00 New Revision: dea08c2b67f38dba707003374f41b2277ab564d4 URL: https://github.com/llvm/llvm-project/commit/dea08c2b67f38dba707003374f41b2277ab564d4 DIFF: https://github.com/llvm/llvm-project/commit/dea08c2b67f38dba707003374f41b2277ab564d4.diff

[clang] [llvm] [Coroutines] Replace coro.outside.frame metadata with an intrinsic (PR #129255)

2025-02-28 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff 71389e565db6c4f9b5b4515baaf711271ed29877 c1fe663ceacd31064b2758e43de1afc3f51d3f2c --e

[clang] Fix RegionStore assertion failure after #127602 (PR #129224)

2025-02-28 Thread Balázs Benics via cfe-commits
https://github.com/balazs-benics-sonarsource closed https://github.com/llvm/llvm-project/pull/129224 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [libcxx] [llvm] [libc++][ranges] P2542R8: Implement `views::concat` (PR #120920)

2025-02-28 Thread Nhat Nguyen via cfe-commits
@@ -0,0 +1,646 @@ +// -*- C++ -*- +//===--===// +// +// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions. +// See https://llvm.org/LICENSE.txt for license information. +// SPDX-License

[clang] [llvm] [win][x64] Unwind v2 3/n: Add support for emitting unwind v2 information (equivalent to MSVC /d2epilogunwind) (PR #129142)

2025-02-28 Thread Pedro Miguel Justo via cfe-commits
pmsjt wrote: > How do epilogs work in chained unwind tables? Do the epilog opcodes from the > original table get ignored? They are additive. When execution is on a fragment, it is assumed that all unwind opcodes from the parent fragment(s), all the way to the principal fragment, are already a

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread via cfe-commits
https://github.com/choikwa created https://github.com/llvm/llvm-project/pull/129347 …__builtin_scalbn Clang generates library calls for __builtin_* functions which can be a problem for GPUs that cannot handle them. This patch generates a call to device implementation for __builtin_logb and an

[clang] [CUDA][HIP] fix virtual dtor host/device attr (PR #128926)

2025-02-28 Thread Yaxun Liu via cfe-commits
@@ -32,22 +32,24 @@ public: template class B; } -// The implicit host/device attrs of virtual dtor B::~B() is inferred to -// have implicit device attr since dtors of its members and parent classes can -// be executed on device. This causes a diagnostic since B::~B() must -//

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread via cfe-commits
choikwa wrote: The motivation came from CUDA cccl code containing __builtin_logb and __builtin_scalbn and being able to handle them. https://github.com/llvm/llvm-project/pull/129347 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists

[clang] [MC/DC] Enable usage of `!` among `&&` and `||` (PR #125406)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni updated https://github.com/llvm/llvm-project/pull/125406 >From a6d4be0e4b05b411c7160385485cfed0f173965c Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 2 Feb 2025 21:55:43 +0900 Subject: [PATCH 1/4] [MC/DC] Update CoverageMapping tests To resolve the error,

[clang] [llvm] [win][x64] Unwind v2 3/n: Add support for emitting unwind v2 information (equivalent to MSVC /d2epilogunwind) (PR #129142)

2025-02-28 Thread Pedro Miguel Justo via cfe-commits
pmsjt wrote: https://learn.microsoft.com/en-us/cpp/build/exception-handling-x64?view=msvc-170#chained-unwind-info-structures I don't know what an "atend" is, but what you are describing doesn't seem to be a possible solution. Epilogs must be present in the main function entry. Only the main fu

[clang] [clang] Pass fp128 indirectly and return in xmm0 on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
https://github.com/tgross35 updated https://github.com/llvm/llvm-project/pull/115052 >From 6afdfd07a22260914b45363870a7be54324bd736 Mon Sep 17 00:00:00 2001 From: Trevor Gross Date: Tue, 5 Nov 2024 07:00:35 -0500 Subject: [PATCH 1/2] [clang] Add fp128 ABI tests for MinGW (NFC) Add a test based

[clang] [clang] Pass fp128 indirectly and return in xmm0 on Windows (PR #115052)

2025-02-28 Thread Trevor Gross via cfe-commits
tgross35 wrote: Thanks for answering that question, I wouldn't have had a good answer outside of consistency. Does mid-level optimizations refer to optimizations done in Clang rather than in LLVM? Somebody will need to land this for me, the two commits should come separate (first is NFC). >

[clang] [Clang] Check for uninitialized use in lambda within CXXOperatorCallExpr (PR #129198)

2025-02-28 Thread via cfe-commits
https://github.com/zhaohuiw42 updated https://github.com/llvm/llvm-project/pull/129198 >From 6667d8b34a2d33b1e0b2f56466cac558d13cbf80 Mon Sep 17 00:00:00 2001 From: zhaohui Date: Fri, 28 Feb 2025 14:12:39 +0800 Subject: [PATCH] [Clang] Check for uninitialized use in lambda within CXXOperatorCa

[clang] [Clang] Check for uninitialized use in lambda within CXXOperatorCallExpr (PR #129198)

2025-02-28 Thread via cfe-commits
@@ -12806,6 +12810,19 @@ namespace { HandleValue(Arg->IgnoreParenImpCasts()); } +void VisitLambdaExpr(LambdaExpr *E) { + if (!isInCXXOperatorCall) { +Inherited::VisitLambdaExpr(E); +return; + } + + for (const auto &init : E->captu

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread Matt Arsenault via cfe-commits
https://github.com/arsenm requested changes to this pull request. clang should never emit a call to an ocml function https://github.com/llvm/llvm-project/pull/129347 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bi

[clang] [Clang] Check for uninitialized use in lambda within CXXOperatorCallExpr (PR #129198)

2025-02-28 Thread via cfe-commits
@@ -12806,6 +12810,19 @@ namespace { HandleValue(Arg->IgnoreParenImpCasts()); } +void VisitLambdaExpr(LambdaExpr *E) { + if (!isInCXXOperatorCall) { +Inherited::VisitLambdaExpr(E); +return; + } + + for (const auto &init : E->captu

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread via cfe-commits
choikwa wrote: I would have preferred to do the overloads in the target specific header file, but __builtin_* methods couldn't be overloaded. I noticed that hip math header already overloads 'double logb(double)' by calling ocml version, so I mirrored that in clang. If there is better approac

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > The motivation came from CUDA cccl code containing __builtin_logb and > __builtin_scalbn and being able to handle them. In my ideal world, unhandled builtins would be lowered before the linker and we'd have a working library called `libm.a` that matches the platform. https://

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread Joseph Huber via cfe-commits
jhuber6 wrote: > I would have preferred to do the overloads in the target specific header > file, but __builtin_* methods couldn't be overloaded. I noticed that hip math > header already overloads 'double logb(double)' by calling ocml version, so I > mirrored that in clang. If there is better

[clang] [clang-tools-extra] [libcxx] [llvm] [libc++][ranges] P2542R8: Implement `views::concat` (PR #120920)

2025-02-28 Thread Nhat Nguyen via cfe-commits
https://github.com/changkhothuychung updated https://github.com/llvm/llvm-project/pull/120920 error: too big or took too long to generate ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[clang] [llvm] [win][x64] Unwind v2 3/n: Add support for emitting unwind v2 information (equivalent to MSVC /d2epilogunwind) (PR #129142)

2025-02-28 Thread Eli Friedman via cfe-commits
efriedma-quic wrote: So if an unwind table has an epilog opcode, and you chain another unwind table to it, the chained unwind table inherits the epilog opcode? You could use that. The primary unwind table ends at the first epilog, you mark it with an "atend" epilog opcode. The next region sta

[clang] [AMDGPU][clang] provide device implementation for __builtin_logb and … (PR #129347)

2025-02-28 Thread via cfe-commits
choikwa wrote: > clang should never emit a call to an ocml function Would IR body of the ocml function be a suitable replacement to emit (for now)? I understand once we have libm/libc implementation, clang can decide between the two. https://github.com/llvm/llvm-project/pull/129347 __

[clang] [MC/DC] Refactor MCDC::State::Decision. NFC. (PR #125408)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni updated https://github.com/llvm/llvm-project/pull/125408 >From d414f29ed8732c77fdcd05cc3b066e9ee0d9de07 Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 2 Feb 2025 21:59:33 +0900 Subject: [PATCH 1/2] [MC/DC] Refactor MCDC::State::Decision. NFC. Introduce `ID

[clang] [MC/DC] Update CoverageMapping tests (PR #125404)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni updated https://github.com/llvm/llvm-project/pull/125404 >From a6d4be0e4b05b411c7160385485cfed0f173965c Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 2 Feb 2025 21:55:43 +0900 Subject: [PATCH 1/2] [MC/DC] Update CoverageMapping tests To resolve the error,

[clang] [MC/DC] Update CoverageMapping tests (PR #125404)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni edited https://github.com/llvm/llvm-project/pull/125404 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] cab738b - [MC/DC] Update CoverageMapping tests (#125404)

2025-02-28 Thread via cfe-commits
Author: NAKAMURA Takumi Date: 2025-03-01T15:58:39+09:00 New Revision: cab738bea1a6d06c6aaebc0e9ad5954a2c5c1e0b URL: https://github.com/llvm/llvm-project/commit/cab738bea1a6d06c6aaebc0e9ad5954a2c5c1e0b DIFF: https://github.com/llvm/llvm-project/commit/cab738bea1a6d06c6aaebc0e9ad5954a2c5c1e0b.dif

[clang] [MC/DC] Update CoverageMapping tests (PR #125404)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni closed https://github.com/llvm/llvm-project/pull/125404 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [MC/DC] Enable usage of `!` among `&&` and `||` (PR #125406)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni edited https://github.com/llvm/llvm-project/pull/125406 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [MC/DC] Handle __builtin_expect as if parenthses (PR #125405)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni edited https://github.com/llvm/llvm-project/pull/125405 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-tools-extra] [libcxx] [llvm] [libc++] constexpr deque (PR #129367)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-libcxx Author: Nhat Nguyen (changkhothuychung) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/129367.diff 1 Files Affected: - (modified) libcxx/include/deque (+6-5) ``diff diff --git a/libcxx/include/deque b/libcxx/incl

[clang] [clang-tools-extra] [libcxx] [llvm] [libc++] constexpr deque (PR #129367)

2025-02-28 Thread Nhat Nguyen via cfe-commits
https://github.com/changkhothuychung closed https://github.com/llvm/llvm-project/pull/129367 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [MC/DC] Handle __builtin_expect as if parenthses (PR #125405)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni updated https://github.com/llvm/llvm-project/pull/125405 >From a6d4be0e4b05b411c7160385485cfed0f173965c Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 2 Feb 2025 21:55:43 +0900 Subject: [PATCH 1/3] [MC/DC] Update CoverageMapping tests To resolve the error,

[clang] [MC/DC] Refactor MCDC::State::Decision. NFC. (PR #125408)

2025-02-28 Thread NAKAMURA Takumi via cfe-commits
https://github.com/chapuni updated https://github.com/llvm/llvm-project/pull/125408 >From a6d4be0e4b05b411c7160385485cfed0f173965c Mon Sep 17 00:00:00 2001 From: NAKAMURA Takumi Date: Sun, 2 Feb 2025 21:55:43 +0900 Subject: [PATCH 1/4] [MC/DC] Update CoverageMapping tests To resolve the error,

[clang] [clang-format] Don't sort includes for C# (PR #129369)

2025-02-28 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: Owen Pan (owenca) Changes Fixes #106194 --- Full diff: https://github.com/llvm/llvm-project/pull/129369.diff 2 Files Affected: - (modified) clang/lib/Format/Format.cpp (+2-1) - (modified) clang/unittests/Format/SortIncludesTest.

[clang] [clang-format] Don't sort includes for C# (PR #129369)

2025-02-28 Thread Owen Pan via cfe-commits
https://github.com/owenca created https://github.com/llvm/llvm-project/pull/129369 Fixes #106194 >From 38f6c5e4509ba3420aeb69661c764d037bc0 Mon Sep 17 00:00:00 2001 From: Owen Pan Date: Fri, 28 Feb 2025 23:50:04 -0800 Subject: [PATCH] [clang-format] Don't sort includes for C# Fixes #10619

[clang-tools-extra] [clang-tidy] detect arithmetic operations within member list initialization in modernize-use-default-member-init check (PR #129370)

2025-02-28 Thread David Rivera via cfe-commits
https://github.com/RiverDave created https://github.com/llvm/llvm-project/pull/129370 This aims to address a portion of #122480 by adding matchers on binary operators. **This allows explicit arithmetic operations within initializers.** ### Note I'm aware of the other false-negatives presented

[clang] [llvm] [libclang/python] Enable packaging libclang bindings (PR #125806)

2025-02-28 Thread Jannick Kremer via cfe-commits
DeinAlptraum wrote: @Endilll @LecrisUT what else do you think needs to be done here to get this approved? https://github.com/llvm/llvm-project/pull/125806 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/l

[clang] [MC/DC] Update CoverageMapping tests (PR #125404)

2025-02-28 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux-bootstrap-asan` running on `sanitizer-buildbot8` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/24/builds/5832 Here is the releva

[clang-tools-extra] [clang-tidy] detect arithmetic operations within member list initialization in modernize-use-default-member-init check (PR #129370)

2025-02-28 Thread David Rivera via cfe-commits
https://github.com/RiverDave edited https://github.com/llvm/llvm-project/pull/129370 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] detect arithmetic operations within member list initialization in modernize-use-default-member-init check (PR #129370)

2025-02-28 Thread David Rivera via cfe-commits
https://github.com/RiverDave edited https://github.com/llvm/llvm-project/pull/129370 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

<    1   2   3   4