[clang] [llvm] [PowerPC] Test SPE incompatibility with VSX (PR #147184)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/147184 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PowerPC] Remove SPE from triple in VSX test (PR #147184)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/147184 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [PowerPC] Remove SPE from triple in VSX test (PR #147184)

https://github.com/guy-david updated https://github.com/llvm/llvm-project/pull/147184 >From 3336929fcd243162d2063998a2624017e4e89143 Mon Sep 17 00:00:00 2001 From: Guy David Date: Sun, 6 Jul 2025 15:22:38 +0300 Subject: [PATCH] [PowerPC] Test SPE incompatibility with VSX PPCSubtarget is not al

[clang] [HLSL][RootSignature] Audit `RootSignatureParser` diagnostic production (PR #147800)

guy-david wrote: Hey, this seems to have broken https://green.lab.llvm.org/job/llvm.org/job/clang-stage1-cmake-RA-expensive/4708/testReport/Clang/SemaHLSL/RootSignature_resource_ranges_err_hlsl/, would you mind looking into it? https://github.com/llvm/llvm-project/pull/147800 _

[clang] [PowerPC] Remove SPE from triple in VSX test (PR #147184)

https://github.com/guy-david created https://github.com/llvm/llvm-project/pull/147184 This test does not fail because the PPCSubtarget was not initialized, and otherwise a fatal error occurs: "SPE and traditional floating point cannot both be enabled.". Tested by adding `-mllvm -enable-matrix`

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

guy-david wrote: > I just got a case: > > ``` > class A { > public: >class B { >public: > B(A *); > > // some non static data fields >}; > >B b(this); > }; > ``` > > Does this a valid prove that this optimization is not valid ? This example is fine because `

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

guy-david wrote: > > @zygoloid Can you explain in your example why `a.n == 2` must be true, when > > your interpretation (which I understood in the same manner) of the > > standard's wording does indicate that the object's state is unspecified? > > My reading is that the standard says that the

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

guy-david wrote: @efriedma-quic ping :) https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david ready_for_review https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

guy-david wrote: Comparison between latest Clang and GCC's output for a snippet out of a benchmark that could use this optimization: https://godbolt.org/z/35EEvcsPr. I've ran llvm-test-suite ten times for the before and after, it executed correctly and expectedly saw no performance gains: ```

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

guy-david wrote: Yeah, I realize it's misplaced, I am not familiar with that part of the project, see the first paragraph in the PR description. I don't really agree with your second point about breaking people's existing assumptions on UB :) I am willing to run correctness suites to further v

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david edited https://github.com/llvm/llvm-project/pull/136792 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Add `noalias` to `this` pointer in C++ constructors (PR #136792)

https://github.com/guy-david created https://github.com/llvm/llvm-project/pull/136792 Note: the patch is probably amending the wrong piece of code, I've tried to add it to `buildThisParam` but hit an assertion because of a missing translation unit context. Clang, unlike GCC, does not transfor