[clang] [clang-format] Google Style: disable DerivePointerAlignment. (PR #149602)

2025-07-19 Thread Owen Pan via cfe-commits
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/149602 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Jacques Pienaar via cfe-commits
https://github.com/jpienaar updated https://github.com/llvm/llvm-project/pull/149148 >From b404f5390ac5684c7452e69f6fe209e5215f8929 Mon Sep 17 00:00:00 2001 From: Jacques Pienaar Date: Wed, 16 Jul 2025 17:37:53 + Subject: [PATCH 1/8] [clang-tidy] Add MLIR check for old op builder usage. Mo

[clang] [clang][bytecode] Use in Expr::tryEvaluateStrLen() (PR #149677)

2025-07-19 Thread Timm Baeder via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: https://github.com/tbaederr edited https://github.com/llvm/llvm-project/pull/149677 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] [Clang-Repl] Add custom function as lambda in launchExecutor and fetch PID of launched executor (PR #147478)

2025-07-19 Thread Abhinav Kumar via cfe-commits
https://github.com/kr-2003 updated https://github.com/llvm/llvm-project/pull/147478 >From fbe4344831538480be33accd35ef618c6d0e50b3 Mon Sep 17 00:00:00 2001 From: kr-2003 Date: Tue, 1 Jul 2025 18:55:21 +0530 Subject: [PATCH 1/9] pipes for redirection in oop jit --- .../clang/Interpreter/Remote

[libclc] [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (PR #149657)

2025-07-19 Thread Michał Górny via cfe-commits
mgorny wrote: Thx. https://github.com/llvm/llvm-project/pull/149657 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [flang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread via cfe-commits
https://github.com/koachan edited https://github.com/llvm/llvm-project/pull/149652 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (PR #149657)

2025-07-19 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `premerge-monolithic-linux` running on `premerge-linux-1` while building `libclc` at step 7 "test-build-unified-tree-check-all". Full details are available at: https://lab.llvm.org/buildbot/#/builders/153/builds/38701 Here i

[libclc] [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (PR #149657)

2025-07-19 Thread Sylvestre Ledru via cfe-commits
https://github.com/sylvestre closed https://github.com/llvm/llvm-project/pull/149657 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] 58c3aff - [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (#149657)

2025-07-19 Thread via cfe-commits
Author: Michał Górny Date: 2025-07-20T12:26:51+09:00 New Revision: 58c3affdaa732fd6f8c7e6640396e7c6366bac9d URL: https://github.com/llvm/llvm-project/commit/58c3affdaa732fd6f8c7e6640396e7c6366bac9d DIFF: https://github.com/llvm/llvm-project/commit/58c3affdaa732fd6f8c7e6640396e7c6366bac9d.diff

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
mintsuki wrote: A bunch of tests fail now... I feel like this change is too invasive for what I can possibly do on my own, but I hope that the original commit message is helpful in describing the problem at hand. https://github.com/llvm/llvm-project/pull/149681

[clang] [clang] Don't warn on zero literals with -std=c2y (PR #149688)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Timothy Herchen (anematode) Changes Fixes #149669; the old check compared with the end of the literal, but we can just check that after parsing digits, we're pointing to one character past the token start. --- Full diff: https://github.c

[clang] [clang] Don't warn on zero literals with -std=c2y (PR #149688)

2025-07-19 Thread Timothy Herchen via cfe-commits
https://github.com/anematode created https://github.com/llvm/llvm-project/pull/149688 Fixes #149669; the old check compared with the end of the literal, but we can just check that after parsing digits, we're pointing to one character past the token start. >From 5def3c282d446e6ac687cdef37710e9

[clang] [flang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread via cfe-commits
https://github.com/koachan updated https://github.com/llvm/llvm-project/pull/149652 >From 84da1b197113869b684f87bf580b37122d6a15e3 Mon Sep 17 00:00:00 2001 From: Koakuma Date: Sat, 19 Jul 2025 05:40:56 +0700 Subject: [PATCH 1/2] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp Th

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
https://github.com/mintsuki updated https://github.com/llvm/llvm-project/pull/149681 >From 491a834787083c74c3584839dc036cc1f54519c1 Mon Sep 17 00:00:00 2001 From: Mintsuki Date: Sun, 20 Jul 2025 00:21:08 +0200 Subject: [PATCH] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid callin

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
mintsuki wrote: > Can you add a test to clang/test/Driver to show the behavior difference ? @MaskRay done. Does this work? https://github.com/llvm/llvm-project/pull/149681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
https://github.com/mintsuki updated https://github.com/llvm/llvm-project/pull/149681 >From 468635d12a26a7e1d38fe9d1321a5f76167ddead Mon Sep 17 00:00:00 2001 From: Mintsuki Date: Sun, 20 Jul 2025 00:21:08 +0200 Subject: [PATCH] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid callin

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread Fangrui Song via cfe-commits
MaskRay wrote: Can you add a test to clang/test/Driver to show the behavior difference ? https://github.com/llvm/llvm-project/pull/149681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Shafik Yaghmour via cfe-commits
@@ -1445,3 +1445,8 @@ static_assert(test_member_null(), ""); } } + +namespace GH149500 { + unsigned int * p = &(*(unsigned int *)0x400); + static const void *q = &(*(const struct sysrq_key_op *)0); shafik wrote: Maybe also `nullptr` test? https://github.co

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Shafik Yaghmour via cfe-commits
@@ -153,3 +150,7 @@ struct PR35214_X { int PR35214_x; int PR35214_y = ((struct PR35214_X *)&PR35214_x)->arr[1]; // expected-error {{not a compile-time constant}} int *PR35214_z = &((struct PR35214_X *)&PR35214_x)->arr[1]; // ok, &PR35214_x + 2 + + +int * GH149500_p = &(*(int

[clang] [llvm] [docs] Add clang/llvm release notes for mipsel-windows-* targets (PR #147133)

2025-07-19 Thread Brad Smith via cfe-commits
brad0 wrote: This should be taken and created as a MR against the 21 release. https://github.com/llvm/llvm-project/pull/147133 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] AMDGPU: Support v_wmma_f32_16x16x128_f8f6f4 on gfx1250 (PR #149684)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Changpeng Fang (changpeng) Changes --- Patch is 132.52 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/149684.diff 31 Files Affected: - (modified) clang/include/clang/Basic/BuiltinsAMDGPU.de

[clang] [llvm] AMDGPU: Support v_wmma_f32_16x16x128_f8f6f4 on gfx1250 (PR #149684)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-mc Author: Changpeng Fang (changpeng) Changes --- Patch is 132.52 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/149684.diff 31 Files Affected: - (modified) clang/include/clang/Basic/BuiltinsAMDGPU.def (

[clang] [llvm] AMDGPU: Support v_wmma_f32_16x16x128_f8f6f4 on gfx1250 (PR #149684)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-backend-amdgpu @llvm/pr-subscribers-llvm-ir Author: Changpeng Fang (changpeng) Changes --- Patch is 132.52 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/149684.diff 31 Files Affected: - (modified) clang

[clang] [llvm] AMDGPU: Support v_wmma_f32_16x16x128_f8f6f4 on gfx1250 (PR #149684)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-llvm-transforms Author: Changpeng Fang (changpeng) Changes --- Patch is 132.52 KiB, truncated to 20.00 KiB below, full version: https://github.com/llvm/llvm-project/pull/149684.diff 31 Files Affected: - (modified) clang/include/clang/Basic/Builtin

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread Brad Smith via cfe-commits
brad0 wrote: cc @MaskRay https://github.com/llvm/llvm-project/pull/149681 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (mintsuki) Changes This change primarily makes it so that when targeting freestanding/unknown OSes, the driver will not use `gcc` to link, but rather will link using the chosen linker (`-fuse-ld=...`) directly. If Clang is to be a cro

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] [Clang][Driver] Override Generic_ELF::buildLinker() to avoid calling gcc to link (PR #149681)

2025-07-19 Thread via cfe-commits
https://github.com/mintsuki created https://github.com/llvm/llvm-project/pull/149681 This change primarily makes it so that when targeting freestanding/unknown OSes, the driver will not use `gcc` to link, but rather will link using the chosen linker (`-fuse-ld=...`) directly. If Clang is to be

[clang] Reland [clang][modules-driver] Add scanner to detect C++20 module presence (PR #147630)

2025-07-19 Thread Naveen Seth Hanig via cfe-commits
naveen-seth wrote: This now also leverages the `Scanner` in `DependencyDirectivesScanner.cpp` to reuse its logic for properly lexing module related directives. For this to happen, I had to first land two fixes with #148674 and #148685. https://github.com/llvm/llvm-project/pull/147630 _

[clang] Reland [clang][modules-driver] Add scanner to detect C++20 module presence (PR #147630)

2025-07-19 Thread Naveen Seth Hanig via cfe-commits
https://github.com/naveen-seth updated https://github.com/llvm/llvm-project/pull/147630 >From 6b440de2ed76a6095c4b68ba1301dfbd19f9a285 Mon Sep 17 00:00:00 2001 From: Naveen Seth Hanig Date: Fri, 13 Jun 2025 09:19:21 +0200 Subject: [PATCH 1/4] Reland [clang][modules-driver] Add scanner to detect

[clang] [clang][docs] Add all clang-tools-extra to 'ClangTools.rst' (PR #148622)

2025-07-19 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor closed https://github.com/llvm/llvm-project/pull/148622 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] 54492c2 - [clang][docs] Add all clang-tools-extra to 'ClangTools.rst' (#148622)

2025-07-19 Thread via cfe-commits
Author: Manas Date: 2025-07-20T01:18:24+03:00 New Revision: 54492c231c5d9091d086bfb767423415ea6bd0bc URL: https://github.com/llvm/llvm-project/commit/54492c231c5d9091d086bfb767423415ea6bd0bc DIFF: https://github.com/llvm/llvm-project/commit/54492c231c5d9091d086bfb767423415ea6bd0bc.diff LOG: [c

[clang-tools-extra] [clang-tidy] Teach `readability-uppercase-literal-suffix` about C++23 and C23 suffixes (PR #148275)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,162 @@ +// TODO: When Clang adds support for decimal floating point types, enable these tests by: +//1. Removing all the #if 0 + #endif guards. +//2. Removing all occurrences of the string "DISABLED-" in this file. +//3. Deleting this message. + +// RUN: %c

[clang-tools-extra] [clang-tidy] Teach `readability-uppercase-literal-suffix` about C++23 and C23 suffixes (PR #148275)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,248 @@ +// TODO: When Clang adds support for C++23 floating-point types, enable these tests by: +//1. Removing all the #if 0 + #endif guards. +//2. Removing all occurrences of the string "DISABLED-" in this file. +//3. Deleting this message. +// These suffi

[clang-tools-extra] [clang-tidy] Teach `readability-uppercase-literal-suffix` about C++23 and C23 suffixes (PR #148275)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -225,6 +226,89 @@ void integer_complex_suffix() { static_assert(v28 == 1J, ""); } +// This is a C++23 feature, but Clang supports it in earlier language modes +// as an extension, so we test it unconditionally. vbvictor wrote: I think we should not depen

[clang-tools-extra] [clang-tidy] Teach `readability-uppercase-literal-suffix` about C++23 and C23 suffixes (PR #148275)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,248 @@ +// TODO: When Clang adds support for C++23 floating-point types, enable these tests by: +//1. Removing all the #if 0 + #endif guards. +//2. Removing all occurrences of the string "DISABLED-" in this file. +//3. Deleting this message. +// These suffi

[clang-tools-extra] [clang-tidy] Teach `readability-uppercase-literal-suffix` about C++23 and C23 suffixes (PR #148275)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,248 @@ +// TODO: When Clang adds support for C++23 floating-point types, enable these tests by: +//1. Removing all the #if 0 + #endif guards. +//2. Removing all occurrences of the string "DISABLED-" in this file. +//3. Deleting this message. +// These suffi

[clang-tools-extra] [clang-tidy] Make `bugprone-unhandled-self-assignment` check more general (PR #147066)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -540,6 +547,89 @@ class NotACopyAssignmentOperator { Uy *getUy() const { return Ptr2; } }; +// Support "extended" copy/move constructors +class AllocatorAwareClass { vbvictor wrote: Could you add cases when the check will fire (I suppose it at least): -

[clang-tools-extra] [clang-tidy] Make `bugprone-unhandled-self-assignment` check more general (PR #147066)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -69,6 +69,28 @@ void UnhandledSelfAssignmentCheck::registerMatchers(MatchFinder *Finder) { cxxMethodDecl(unless(hasDescendant(cxxMemberCallExpr(callee(cxxMethodDecl( hasName("operator="), ofClass(equalsBoundNode("class"; + // Checking that some ki

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,51 @@ +// RUN: %check_clang_tidy --match-partial-fixes %s llvm-mlir-op-builder %t + +namespace mlir { +class Location {}; +class OpBuilder { +public: + template + OpTy create(Location location, Args &&...args) { +return OpTy(args...); + } + Location getUnknownL

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,103 @@ +//===--- MLIROpBuilderCheck.cpp - clang-tidy --===// +// +// 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-Identifier: Ap

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,21 @@ +.. title:: clang-tidy - llvm-mlir-op-builder + +llvm-mlir-op-builder + + +Flags usage of old form of invoking create on MLIR's ``OpBuilder`` and suggests +new form. vbvictor wrote: I think we should write why the user should

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,103 @@ +//===--- MLIROpBuilderCheck.cpp - clang-tidy --===// +// +// 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-Identifier: Ap

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor edited https://github.com/llvm/llvm-project/pull/149148 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -40,6 +41,7 @@ class LLVMModule : public ClangTidyModule { CheckFactories.registerCheck( "llvm-qualified-auto"); CheckFactories.registerCheck("llvm-twine-local"); +CheckFactories.registerCheck("llvm-mlir-op-builder"); vbvictor wrote: Ple

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
@@ -0,0 +1,29 @@ +//===--- MLIROpBuilderCheck.h - clang-tidy --*- 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-Identifier: Apa

[clang-tools-extra] [clang-tidy] Add MLIR check for old op builder usage. (PR #149148)

2025-07-19 Thread Baranov Victor via cfe-commits
https://github.com/vbvictor commented: I think we need a more verbose check name for users to understand what is the check about, e.g. `llvm-prefer-new-mlir-op-builder` `llvm-use-new-mlir-op-builder` `llvm-use-static-function-mlir-op-builder` https://github.com/llvm/llvm-project/pull/149148 ___

[clang] [clang][bytecode] Use in Expr::tryEvaluateStrLen() (PR #149677)

2025-07-19 Thread via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: llvmbot wrote: @llvm/pr-subscribers-clang Author: Timm Baeder (tbaederr) Changes --- Full diff: https://github.com/llvm/llvm-project/pull/149677.diff 4 Files Affected: - (modified) clang/lib/AST/ByteCode/Context.cpp (+37) - (modified

[clang] [clang][bytecode] Use in Expr::tryEvaluateStrLen() (PR #149677)

2025-07-19 Thread Timm Baeder via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: https://github.com/tbaederr created https://github.com/llvm/llvm-project/pull/149677 None >From 9db49be1f605ec039b984bd7df3e29c127df1866 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Sat, 19 Jul 2025 23:02:13 +0200 Subject

[clang] [CIR] Add support for binary operations on bitfield members (PR #149676)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clangir Author: None (Andres-Salamanca) Changes This PR introduces support for binary operations on bitfield members. --- Full diff: https://github.com/llvm/llvm-project/pull/149676.diff 2 Files Affected: - (modified) clang/lib/CIR/CodeGen/CIRGenExpr

[clang] [CIR] Add support for binary operations on bitfield members (PR #149676)

2025-07-19 Thread via cfe-commits
https://github.com/Andres-Salamanca created https://github.com/llvm/llvm-project/pull/149676 This PR introduces support for binary operations on bitfield members. >From c5da343296558a3a418c6282b104196d20f88a60 Mon Sep 17 00:00:00 2001 From: Andres Salamanca Date: Sat, 19 Jul 2025 15:54:56 -050

[clang] fix-149477-- Emit correct closure members for lambdas in CreateLimitedType (PR #149674)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: Ayush Pareek (ayushpareek2003) Changes Fix an issue where local lambdas caused a crash when importing the std module in LLDB due to missing capture fields in debug info. Now closures always emit all fields for lambdas by skipping the `is

[clang] fix-149477-- Emit correct closure members for lambdas in CreateLimitedType (PR #149674)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-codegen Author: Ayush Pareek (ayushpareek2003) Changes Fix an issue where local lambdas caused a crash when importing the std module in LLDB due to missing capture fields in debug info. Now closures always emit all fields for lambdas by skipping

[clang] fix-149477-- Emit correct closure members for lambdas in CreateLimitedType (PR #149674)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-debuginfo Author: Ayush Pareek (ayushpareek2003) Changes Fix an issue where local lambdas caused a crash when importing the std module in LLDB due to missing capture fields in debug info. Now closures always emit all fields for lambdas by skipping the

[clang] fix-149477-- Emit correct closure members for lambdas in CreateLimitedType (PR #149674)

2025-07-19 Thread via cfe-commits
github-actions[bot] wrote: Thank you for submitting a Pull Request (PR) to the LLVM Project! This PR will be automatically labeled and the relevant teams will be notified. If you wish to, you can add reviewers by using the "Reviewers" section on this page. If this is not working for you, it

[clang] fix-149477-- Emit correct closure members for lambdas in CreateLimitedType (PR #149674)

2025-07-19 Thread Ayush Pareek via cfe-commits
https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/149674 Fix an issue where local lambdas caused a crash when importing the std module in LLDB due to missing capture fields in debug info. Now closures always emit all fields for lambdas by skipping the `isImp

[libclc] [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (PR #149657)

2025-07-19 Thread Sam James via cfe-commits
https://github.com/thesamesam approved this pull request. https://github.com/llvm/llvm-project/pull/149657 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang][bytecode] Diagnose dereferencing a null pointer (PR #149330)

2025-07-19 Thread LLVM Continuous Integration via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-s390x-linux` running on `systemz-1` while building `clang` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/88/builds/14

[clang] [HIP][Clang][Driver] Move BC preference logic into ROCm detection (PR #149294)

2025-07-19 Thread Yaxun Liu via cfe-commits
@@ -77,6 +79,82 @@ class RocmInstallationDetector { SPACKReleaseStr(SPACKReleaseStr.str()) {} }; + struct CommonBitcodeLibsPreferences { +CommonBitcodeLibsPreferences(const Driver &D, + const llvm::opt::ArgList &DriverArgs, +

[clang] [clang][bytecode] Diagnose dereferencing a null pointer (PR #149330)

2025-07-19 Thread Timm Baeder via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: https://github.com/tbaederr closed https://github.com/llvm/llvm-project/pull/149330 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fed6db4 - [clang][bytecode] Diagnose dereferencing a null pointer (#149330)

2025-07-19 Thread via cfe-commits
Author: Timm Baeder Date: 2025-07-19T21:39:06+02:00 New Revision: fed6db45dfbe10cf1a3dab663e8ceb7e7974d713 URL: https://github.com/llvm/llvm-project/commit/fed6db45dfbe10cf1a3dab663e8ceb7e7974d713 DIFF: https://github.com/llvm/llvm-project/commit/fed6db45dfbe10cf1a3dab663e8ceb7e7974d713.diff L

[clang] [clang][bytecode] Diagnose dereferencing a null pointer (PR #149330)

2025-07-19 Thread Timm Baeder via cfe-commits
Timm =?utf-8?q?Bäder?= Message-ID: In-Reply-To: https://github.com/tbaederr updated https://github.com/llvm/llvm-project/pull/149330 >From 452950ede23a7a21ca88db3600297d67935ddf51 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Timm=20B=C3=A4der?= Date: Thu, 17 Jul 2025 17:04:18 +0200 Subject: [PAT

[clang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread Fangrui Song via cfe-commits
MaskRay wrote: Add a fclang test? https://github.com/llvm/llvm-project/pull/149652 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-repl] Lay the basic infrastructure for pretty printing of types (PR #148701)

2025-07-19 Thread Vassil Vassilev via cfe-commits
vgvassilev wrote: https://github.com/llvm/llvm-project/commit/0a463bd43e626695b7221b0cf20cdaa5970cfe98 should bring piece in the galaxy. However, I am puzzled by the system-z (s390) -- it is a big endian -- it is a good test bed for the conformance for low-level primitives such as `clang::Val

[clang] 0a463bd - [clang-repl] Mark asan and systemz as unsupported for now.

2025-07-19 Thread Vassil Vassilev via cfe-commits
Author: Vassil Vassilev Date: 2025-07-19T17:45:56Z New Revision: 0a463bd43e626695b7221b0cf20cdaa5970cfe98 URL: https://github.com/llvm/llvm-project/commit/0a463bd43e626695b7221b0cf20cdaa5970cfe98 DIFF: https://github.com/llvm/llvm-project/commit/0a463bd43e626695b7221b0cf20cdaa5970cfe98.diff LO

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread via cfe-commits
llvmbot wrote: Failed to cherry-pick: c43f828 https://github.com/llvm/llvm-project/actions/runs/16391072188 Please manually backport the fix and push it to your github fork. Once this is done, please create a [pull request](https://github.com/llvm/llvm-project/compare) https://github.com/l

[clang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread via cfe-commits
koachan wrote: > > This is so that it's performed globally and not just for clang. > > Can you elaborate what is "globally"? Ah, I meant for flang and other frontends too, not just clang, as per the bug report [here](https://github.com/llvm/llvm-project/issues/138494). https://github.com/llvm

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Corentin Jabot via cfe-commits
cor3ntin wrote: /cherry-pick c43f828 https://github.com/llvm/llvm-project/pull/149648 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin milestoned https://github.com/llvm/llvm-project/pull/149648 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] c43f828 - [Clang] Be less strict about diagnosing null pointer dereference. (#149648)

2025-07-19 Thread via cfe-commits
Author: Corentin Jabot Date: 2025-07-19T19:30:16+02:00 New Revision: c43f828d59672b4844a7409e4660b9f8f509da35 URL: https://github.com/llvm/llvm-project/commit/c43f828d59672b4844a7409e4660b9f8f509da35 DIFF: https://github.com/llvm/llvm-project/commit/c43f828d59672b4844a7409e4660b9f8f509da35.diff

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin closed https://github.com/llvm/llvm-project/pull/149648 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] fix: replace report_fatal_error with Diags and exit (PR #147959)

2025-07-19 Thread Fangrui Song via cfe-commits
@@ -30,11 +30,16 @@ SanitizerSpecialCaseList::create(const std::vector &Paths, std::unique_ptr SanitizerSpecialCaseList::createOrDie(const std::vector &Paths, - llvm::vfs::FileSystem &VFS) { + llvm::vfs

[clang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread Fangrui Song via cfe-commits
MaskRay wrote: > This is so that it's performed globally and not just for clang. Can you elaborate what is "globally"? https://github.com/llvm/llvm-project/pull/149652 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-

[clang-tools-extra] [clang-tidy] add modernize-use-constexpr check (PR #146553)

2025-07-19 Thread Julian Schmidt via cfe-commits
5chmidti wrote: > > Then I will split it up into {[11],[14, 17],[20],[23, 26]} > > BTW does this support C23? That would be nice to have too. I totally forgot to add that :) I'll take a look at the rules for it, but might only add it later https://github.com/llvm/llvm-project/pull/146553 ___

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Corentin Jabot via cfe-commits
https://github.com/cor3ntin updated https://github.com/llvm/llvm-project/pull/149648 >From 5cbf868adba8682cc20e4647085ce6361d61455e Mon Sep 17 00:00:00 2001 From: Corentin Jabot Date: Sat, 19 Jul 2025 12:38:20 +0200 Subject: [PATCH 1/2] [Clang] Be less strict about diagnosing null pointer dere

[clang] Thread safety analysis: Allocate FactEntrys with BumpPtrAllocator (PR #149660)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-analysis Author: Aaron Puchert (aaronpuchert) Changes The FactManager managing the FactEntrys stays alive for the analysis of a single function. We are already using that by allocating TIL S-expressions via BumpPtrAllocator. We can do the same wi

[clang] Thread safety analysis: Allocate FactEntrys with BumpPtrAllocator (PR #149660)

2025-07-19 Thread Aaron Puchert via cfe-commits
https://github.com/aaronpuchert created https://github.com/llvm/llvm-project/pull/149660 The FactManager managing the FactEntrys stays alive for the analysis of a single function. We are already using that by allocating TIL S-expressions via BumpPtrAllocator. We can do the same with FactEntrys

[clang] [clang] improve consistency with GCC vector comparison (PR #148954)

2025-07-19 Thread Imple Lee via cfe-commits
ImpleLee wrote: Now I make a utility function `getGCCCompatibleIntTypeForBitwidth` for this type preference. It should also be used for `mode` attribute. There is currently no test about this, but I don't know how to or even whether should I write relevant tests. There are many other places i

[libclc] [libclc] Make library output directories explicit (PR #146833)

2025-07-19 Thread Michał Górny via cfe-commits
mgorny wrote: This change broke installing symlinks. Now all symlinks are installed pointing to the build directory, i.e.: ``` -rw-r--r-- 1 root root 13M Jul 19 16:18 amdgcn--amdhsa.bc lrwxrwxrwx 1 root root 82 Jul 19 16:18 bonaire-amdgcn--.bc -> /var/tmp/portage/llvm-core/libclc-22.0.0.999

[clang] [clang] improve consistency with GCC vector comparison (PR #148954)

2025-07-19 Thread Imple Lee via cfe-commits
https://github.com/ImpleLee ready_for_review https://github.com/llvm/llvm-project/pull/148954 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang] improve consistency with GCC vector comparison (PR #148954)

2025-07-19 Thread Imple Lee via cfe-commits
https://github.com/ImpleLee updated https://github.com/llvm/llvm-project/pull/148954 >From 3b19742073d1625bbf5bb997325dac2f5003e17d Mon Sep 17 00:00:00 2001 From: Imple Lee <80144331+imple...@users.noreply.github.com> Date: Wed, 16 Jul 2025 04:23:45 +0800 Subject: [PATCH 1/3] [clang] improve con

[clang-tools-extra] [clang-tidy] add modernize-use-constexpr check (PR #146553)

2025-07-19 Thread Sean McBride via cfe-commits
seanm wrote: > Then I will split it up into {[11],[14, 17],[20],[23, 26]} BTW does this support C23? That would be nice to have too. https://github.com/llvm/llvm-project/pull/146553 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://list

[libclc] [libclc] Move CMake for prepare_builtins to a subdirectory (PR #148815)

2025-07-19 Thread Michał Górny via cfe-commits
mgorny wrote: Filed #149657 to fix it. https://github.com/llvm/llvm-project/pull/148815 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[libclc] [libclc] Expose `prepare_builtins_*` variables in top-level CMakeLists (PR #149657)

2025-07-19 Thread Michał Górny via cfe-commits
https://github.com/mgorny created https://github.com/llvm/llvm-project/pull/149657 Fix `libclc/utils/CMakeLists.txt` to expose `prepare_builtins_*` variables in parent scope. This was a regression introduced in #148815 where the code was moved into subdirectory, and the variables would no lon

[libclc] [libclc] Move CMake for prepare_builtins to a subdirectory (PR #148815)

2025-07-19 Thread Michał Górny via cfe-commits
mgorny wrote: Obviously because `set` in a subdirectory doesn't get exposed to parent `CMakeLists.txt`… https://github.com/llvm/llvm-project/pull/148815 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/lis

[libclc] [libclc] Move CMake for prepare_builtins to a subdirectory (PR #148815)

2025-07-19 Thread Michał Górny via cfe-commits
mgorny wrote: This broke the standalone builds on Gentoo Linux amd64: ``` [1566/1676] cd /var/tmp/portage/llvm-core/libclc-22.0.0./work/libclc_build && -o /var/tmp/portage/llvm-core/libclc-22.0.0./work/libclc_build/clspv--.bc /var/tmp/portage/llvm-core/libclc-22.0.0./work/libclc_b

[clang-tools-extra] [clang-tidy] add modernize-use-constexpr check (PR #146553)

2025-07-19 Thread via cfe-commits
EugeneZelenko wrote: Please rebase from `main`. Release Notes were cleared after 21 branch. https://github.com/llvm/llvm-project/pull/146553 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com

[clang] [llvm] [Clang] Add `__builtin_stack_address` (PR #148281)

2025-07-19 Thread via cfe-commits
@@ -121,7 +121,7 @@ enum NodeType { /// function calling this intrinsic. SPONENTRY, - /// STACKADDR - Represents the llvm.stackaddr intrinsic. Takes no argument + /// STACKADDR - Represents the llvm.stackaddress intrinsic. Takes no argument moorabbit wro

[clang] [llvm] [Clang] Add `__builtin_stack_address` (PR #148281)

2025-07-19 Thread via cfe-commits
https://github.com/moorabbit updated https://github.com/llvm/llvm-project/pull/148281 >From 029d9fce6cdb75ea4819a107c7ed9f074bb36678 Mon Sep 17 00:00:00 2001 From: moorabbit <215698969+moorab...@users.noreply.github.com> Date: Mon, 7 Jul 2025 09:25:46 -0400 Subject: [PATCH 1/4] [Clang] Add `__bu

[clang] 50408ee - Revert "[clang-repl] Spell out the enum types to appease some bots."

2025-07-19 Thread Vassil Vassilev via cfe-commits
Author: Vassil Vassilev Date: 2025-07-19T14:31:19Z New Revision: 50408eeff6020061ceb6685448e214f36c75f71b URL: https://github.com/llvm/llvm-project/commit/50408eeff6020061ceb6685448e214f36c75f71b DIFF: https://github.com/llvm/llvm-project/commit/50408eeff6020061ceb6685448e214f36c75f71b.diff LO

[clang-tools-extra] [clang-tidy] add modernize-use-constexpr check (PR #146553)

2025-07-19 Thread Julian Schmidt via cfe-commits
5chmidti wrote: > Gave this a very rought look, so only a couple of general > questions/suggestions. > > Could we do something with methods that repeat in some of the visitors, e.g. > `bool TraverseType(QualType QT)` and others. Maybe macros..? I can't think of > a better idea right now. I w

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Timm Baeder via cfe-commits
https://github.com/tbaederr approved this pull request. Other than the small nit, LGTM https://github.com/llvm/llvm-project/pull/149648 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Be less strict about diagnosing null pointer dereference. (PR #149648)

2025-07-19 Thread Timm Baeder via cfe-commits
@@ -9346,9 +9346,12 @@ bool LValueExprEvaluator::VisitUnaryDeref(const UnaryOperator *E) { // [C++26][expr.unary.op] // If the operand points to an object or function, the result // denotes that object or function; otherwise, the behavior is undefined. - return Success

[clang-tools-extra] [clang-tidy] add modernize-use-constexpr check (PR #146553)

2025-07-19 Thread Julian Schmidt via cfe-commits
5chmidti wrote: Fixed multiple vars declared in one statement https://github.com/llvm/llvm-project/pull/146553 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-driver Author: Koakuma (koachan) Changes This is so that it's performed globally and not just for clang. This should fix https://github.com/llvm/llvm-project/issues/138494. --- Full diff: https://github.com/llvm/llvm-project/pull/149652.diff 4

[clang] [SPARC][Driver] Move feature mode selection to Arch/Sparc.cpp (PR #149652)

2025-07-19 Thread via cfe-commits
https://github.com/koachan created https://github.com/llvm/llvm-project/pull/149652 This is so that it's performed globally and not just for clang. This should fix https://github.com/llvm/llvm-project/issues/138494. >From 84da1b197113869b684f87bf580b37122d6a15e3 Mon Sep 17 00:00:00 2001 From:

[clang] [clang-format] Google Style: disable DerivePointerAlignment. (PR #149602)

2025-07-19 Thread James Y Knight via cfe-commits
https://github.com/jyknight edited https://github.com/llvm/llvm-project/pull/149602 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [CIR][NFC] Replace bool by cir::UnaryOpKind in emitComplexPrePostIncDec (PR #149566)

2025-07-19 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `lldb-arm-ubuntu` running on `linaro-lldb-arm-ubuntu` while building `clang` at step 6 "test". Full details are available at: https://lab.llvm.org/buildbot/#/builders/18/builds/19241 Here is the relevant piece of the build lo

[clang] [CIR][NFC] Replace bool by cir::UnaryOpKind in emitComplexPrePostIncDec (PR #149566)

2025-07-19 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux` running on `sanitizer-buildbot7` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/51/builds/20125 Here is the relevant piece of the

[clang] [LifetimeSafety] Add language option for experimental lifetime safety (PR #149592)

2025-07-19 Thread Gábor Horváth via cfe-commits
https://github.com/Xazax-hun approved this pull request. https://github.com/llvm/llvm-project/pull/149592 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

  1   2   >