@@ -1396,20 +1396,27 @@ def HasVendorXMIPSCMov
AssemblerPredicate<(all_of FeatureVendorXMIPSCMov),
"'Xmipscmov' ('mips.ccmov' instruction)">;
def UseCCMovInsn : Predicate<"Subtarget->useCCMovInsn()">;
+
def FeatureVendorXMIPSLSP
: RISCVExten
@@ -125,10 +125,25 @@ class Mips_prefetch_ri
let Inst{6-0} = OPC_CUSTOM_0.Value;
}
+// MIPS Ext Insns Fromat.
ukalappa-mips wrote:
Updated accordingly ..
https://github.com/llvm/llvm-project/pull/155747
___
cfe-c
@@ -121,7 +121,8 @@ def MIPS_P8700 : RISCVProcessorModel<"mips-p8700",
FeatureStdExtZicsr,
FeatureVendorXMIPSCMov,
FeatureVendorXMIPSLSP,
-
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/9] Added the mips extension instructions like ehb,ihb and
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/8] Added the mips extension instructions like ehb,ihb and
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/7] Added the mips extension instructions like ehb,ihb and
YexuanXiao wrote:
> So we're making `__SIZE_TYPE__` be some magic type that you cannot name, only
> procure out of thin air through using sizeof? That doesn't seem like a good
> idea to me.
Within standard scope, it generally does not pose any risks unless X-macros are
used. I have actually s
@@ -165,3 +165,124 @@ void LValueToRValueBitCast_dumps(void *p, char
(*array)[8]) {
unsigned long ptr_arithmetic(void *p) {
return __builtin_bit_cast(unsigned long, p) + 1; // no-crash
}
+
+
+void escape(int*);
+
+struct AllocOpaqueFlag {};
+
+void* operator new(unsigned lon
https://github.com/fmayer approved this pull request.
https://github.com/llvm/llvm-project/pull/154902
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/alejandro-alvarez-sonarsource updated
https://github.com/llvm/llvm-project/pull/155855
From a206f1e24596554f2d7221b795c1196e01d74a5b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20=C3=81lvarez=20Ayll=C3=B3n?=
Date: Mon, 25 Aug 2025 17:02:59 +0200
Subject: [PATCH 1/4] C
@@ -165,3 +165,124 @@ void LValueToRValueBitCast_dumps(void *p, char
(*array)[8]) {
unsigned long ptr_arithmetic(void *p) {
return __builtin_bit_cast(unsigned long, p) + 1; // no-crash
}
+
+
+void escape(int*);
+
+struct AllocOpaqueFlag {};
+
+void* operator new(unsigned lon
@@ -165,3 +165,124 @@ void LValueToRValueBitCast_dumps(void *p, char
(*array)[8]) {
unsigned long ptr_arithmetic(void *p) {
return __builtin_bit_cast(unsigned long, p) + 1; // no-crash
}
+
+
+void escape(int*);
+
+struct AllocOpaqueFlag {};
+
+void* operator new(unsigned lon
@@ -74,6 +74,26 @@ class PointerArithChecker
REGISTER_MAP_WITH_PROGRAMSTATE(RegionState, const MemRegion *, AllocKind)
+namespace {
alejandro-alvarez-sonarsource wrote:
Replaced with `static`
https://github.com/llvm/llvm-project/pull/155855
jrtc27 wrote:
> > If you want __size_t you should just set it to __size_t. I don't see what
> > the point of this approach is other than risking causing issues due to it
> > not just being a plain specifier qualifier list.
>
> Because __size_t is not a keyword, it cannot be directly accessed t
https://github.com/alejandro-alvarez-sonarsource updated
https://github.com/llvm/llvm-project/pull/155855
From a206f1e24596554f2d7221b795c1196e01d74a5b Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alejandro=20=C3=81lvarez=20Ayll=C3=B3n?=
Date: Mon, 25 Aug 2025 17:02:59 +0200
Subject: [PATCH 1/3] C
YexuanXiao wrote:
> If you want __size_t you should just set it to __size_t. I don't see what the
> point of this approach is other than risking causing issues due to it not
> just being a plain specifier qualifier list.
Because __size_t is not a keyword, it cannot be directly accessed through
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/155548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nathan Ridge
Date: 2025-08-29T02:13:17-04:00
New Revision: 7bf92f2827368d452bc979848bb7fb84bc6d449c
URL:
https://github.com/llvm/llvm-project/commit/7bf92f2827368d452bc979848bb7fb84bc6d449c
DIFF:
https://github.com/llvm/llvm-project/commit/7bf92f2827368d452bc979848bb7fb84bc6d449c.diff
HighCommander4 wrote:
> PTAL, I edited the comment. The fact that the number of tests is small
> doesn't really relate to the costs of having a distinct test target, it has
> more to do with the size of the dependency graph.
Makes sense -- thanks!
https://github.com/llvm/llvm-project/pull/155
jrtc27 wrote:
If you want __size_t you should just set it to __size_t. I don't see what the
point of this approach is other than risking causing issues due to it not just
being a plain specifier qualifier list.
https://github.com/llvm/llvm-project/pull/155979
__
@@ -3558,6 +3558,85 @@ struct FormatStyle {
/// \version 9
std::vector NamespaceMacros;
+ /// Control over each component in a numeric literal.
+ enum NumericLiteralComponentStyle : int8_t {
+/// Leave this component of the literal as is.
+NLCS_Leave,
+/// Fo
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (YexuanXiao)
Changes
Previously, I made the `sizeof` expression return the `__size_t` type instead
of `unsigned int`/`long`/`long long` (#143653). However, the types of
the predefined macros `__SIZE_TYPE__` and `__PTRDIFF_TYPE__` obt
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-solaris11-sparcv9`
running on `solaris11-sparcv9` while building `clang` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/13/builds/9296
Here is the relevant piece of t
https://github.com/YexuanXiao created
https://github.com/llvm/llvm-project/pull/155979
Previously, I made the `sizeof` expression return the `__size_t` type instead
of `unsigned int`/`long`/`long long` (#143653). However, the types of the
predefined macros `__SIZE_TYPE__` and `__PTRDIFF_TYPE__
@@ -4148,6 +4148,34 @@ static Value *upgradeX86IntrinsicCall(StringRef Name,
CallBase *CI, Function *F,
Value *Args[] = {CI->getArgOperand(0), CI->getArgOperand(1),
CI->getArgOperand(2)};
+
+// Input arguments types were incorrectly set to vectors
BaiXilin wrote:
> Don't the vpdpwss intrinsics need fixing to use vXi16 types as well?
Yes, VNNI intrinsic `vpdpbsud[,s], vpdpbssd[,s], vpdpbuud[,s], vpdpwssd,
vpdpwuud, vpdpwsud, vpdpwssd` are suffering from the same type problem.
Only `vpdpbusd` and `vpdpbusds` are fixed because:
1. I want t
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-armv8-quick` running
on `linaro-clang-armv8-quick` while building `clang` at step 5 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/154/builds/20856
Here is the relevant piece o
tbaederr wrote:
`[clang][bytecode]` in the title please.
Does this also work with the immediate shifts from
https://github.com/llvm/llvm-project/pull/155542?
https://github.com/llvm/llvm-project/pull/155891
___
cfe-commits mailing list
cfe-commits@li
@@ -3490,7 +3490,15 @@ inline bool AllocN(InterpState &S, CodePtr OpPC,
PrimType T, const Expr *Source,
S.Stk.push(0, nullptr);
return true;
}
- assert(NumElements.isPositive());
+ if (!NumElements.isPositive()) {
+if (!IsNoThrow) {
+ S.FFDiag(Source, diag
@@ -1069,6 +1069,30 @@ namespace BaseCompare {
static_assert(foo());
}
+
+namespace NegativeArraySize {
+
+constexpr void f() { // both-error {{constexpr function never produces a
constant expression}}
+ int x = -1;
+ int *p = new int[x]; //both-note {{cannot allocate ar
@@ -3490,7 +3490,15 @@ inline bool AllocN(InterpState &S, CodePtr OpPC,
PrimType T, const Expr *Source,
S.Stk.push(0, nullptr);
return true;
}
- assert(NumElements.isPositive());
+ if (!NumElements.isPositive()) {
tbaederr wrote:
```suggestion
i
HaohaiWen wrote:
Any other comments?
https://github.com/llvm/llvm-project/pull/123870
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-quick`
running on `linaro-clang-aarch64-quick` while building `clang` at step 5 "ninja
check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/65/builds/21930
Here is the relevant pie
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-sles-build-only` running on `rocm-worker-hw-04-sles` while
building `clang` at step 6 "Add check check-clang".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/140/builds/29768
Here is
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `llvm-clang-x86_64-sie-win`
running on `sie-win-worker` while building `clang` at step 4 "clean-build-dir".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/46/builds/22501
Here is the relevant piece of
Author: Craig Topper
Date: 2025-08-28T22:45:32-07:00
New Revision: 562c27e478744322ea5475a5f135cc030dce799e
URL:
https://github.com/llvm/llvm-project/commit/562c27e478744322ea5475a5f135cc030dce799e
DIFF:
https://github.com/llvm/llvm-project/commit/562c27e478744322ea5475a5f135cc030dce799e.diff
https://github.com/4vtomat approved this pull request.
LGTM~
https://github.com/llvm/llvm-project/pull/155710
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -52,6 +52,15 @@
// CHECK-INTERCHANGE-LOOPS: "-floop-interchange"
// CHECK-NO-INTERCHANGE-LOOPS: "-fno-loop-interchange"
+// RUN: %clang -### -S -fexperimental-loop-fusion %s 2>&1 | FileCheck
-check-prefix=CHECK-FUSE-LOOPS %s
madhur13490 wrote:
I don't thi
@@ -164,7 +170,7 @@ class InterpState final : public State, public SourceMapper
{
/// Reference to the offset-source mapping.
SourceMapper *M;
/// Allocator used for dynamic allocations performed via the program.
- DynamicAllocator Alloc;
+ std::unique_ptr Alloc;
-
@@ -43,6 +43,7 @@ CODEGENOPT(StackArrays, 1, 0) ///< -fstack-arrays (enable the
stack-arrays pass)
CODEGENOPT(VectorizeLoop, 1, 0) ///< Enable loop vectorization.
CODEGENOPT(VectorizeSLP, 1, 0) ///< Enable SLP vectorization.
CODEGENOPT(InterchangeLoops, 1, 0) ///< Enable loop
https://github.com/madhur13490 updated
https://github.com/llvm/llvm-project/pull/142686
>From be9eb6a39906fac945ea206eec80f4cc18bd4896 Mon Sep 17 00:00:00 2001
From: Sebastian Pop
Date: Thu, 22 May 2025 13:50:38 +
Subject: [PATCH 1/8] add -floop-fuse to clang and flang
---
clang/include/c
mtrofin wrote:
> > lgtm, just please remove the unrelated fix in MLInlineAdvisor (the
> > follow-up PR, if you want to do it, doesn't need anything stuffy - just
> > label the title as "[NFC]" - because it's not changing any behavior - and
> > just say something as simple as "made a member sta
@@ -0,0 +1,56 @@
+// -*- 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-
https://github.com/yingcong-wu updated
https://github.com/llvm/llvm-project/pull/154902
>From 57ebc553828bf696b4bde30789e609ba96e94d6d Mon Sep 17 00:00:00 2001
From: Wu Yingcong
Date: Fri, 22 Aug 2025 15:16:47 +0800
Subject: [PATCH 01/13] fix missing by one bug
---
libunwind/src/DwarfParser.h
https://github.com/mtrofin updated
https://github.com/llvm/llvm-project/pull/154541
>From 4bc8f7c83dfd53cc1984d722e78fbc1f19fbf991 Mon Sep 17 00:00:00 2001
From: ae2f
Date: Wed, 20 Aug 2025 22:26:09 +0900
Subject: [PATCH 1/6] Fix of [Static Initialization Order Fiasco
Issue](https://github.com
mtrofin wrote:
> For FeatureMap, I'm not sure if it's supposed to be a plugin mechanism, or it
> can just be a member variable MLInlineAdvisor. The comments don't really
> explain why it's exposed in the header.
@svkeerthy - @efriedma-quic is right, we should probably just make a local copy
o
yingcong-wu wrote:
As this test result shows, the test case also fails without my change to
libunwind:
https://github.com/llvm/llvm-project/actions/runs/17313128556/job/49152917070?pr=154902
(head is be13b4d8).
```
# .---command stderr
# | MemorySanitizer:DEADLYSIGNAL
# | ==1
https://github.com/topperc updated
https://github.com/llvm/llvm-project/pull/155710
>From 41c79fa197820c96637f03c8c2f9cdcaeb3ff5cc Mon Sep 17 00:00:00 2001
From: Craig Topper
Date: Wed, 27 Aug 2025 15:36:08 -0700
Subject: [PATCH 1/2] [RISCV] Verify vfwmaccbf16 and vfncvtbf16 FRM argument in
Se
@@ -692,10 +694,13 @@ std::unique_ptr buildASTFromCodeWithArgs(
StringRef Code, const std::vector &Args, StringRef FileName,
StringRef ToolName, std::shared_ptr
PCHContainerOps,
ArgumentsAdjuster Adjuster, const FileContentMappings &VirtualMappedFiles,
-Diagnos
Himadhith wrote:
> @Himadhith Please add `-ppc-asm-full-reg-names --ppc-vsr-nums-as-vr` to the
> run line.
Done
https://github.com/llvm/llvm-project/pull/151971
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/ma
@@ -166,7 +181,7 @@ def : Pat<(select (XLenVT GPR:$rs2), (XLenVT GPR:$rs1),
(XLenVT GPR:$rs3)),
}
let Predicates = [HasVendorXMIPSLSP], hasSideEffects = 0,
- DecoderNamespace = "Xmipslsp" in {
+ DecoderNamespace = "XMIPS" in {
@@ -1396,20 +1396,27 @@ def HasVendorXMIPSCMov
AssemblerPredicate<(all_of FeatureVendorXMIPSCMov),
"'Xmipscmov' ('mips.ccmov' instruction)">;
def UseCCMovInsn : Predicate<"Subtarget->useCCMovInsn()">;
+
def FeatureVendorXMIPSLSP
: RISCVExten
@@ -146,7 +161,7 @@ let Predicates = [HasVendorXMIPSCBOP] in {
}
let Predicates = [HasVendorXMIPSCMov], hasSideEffects = 0, mayLoad = 0,
mayStore = 0,
- DecoderNamespace = "Xmipscmov" in {
+ DecoderNamespace = "XMIPS" in {
topp
https://github.com/owenca commented:
Final nits.
https://github.com/llvm/llvm-project/pull/151590
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -41,7 +61,16 @@ template
void expectResolution(llvm::StringRef Code, ResolveFnT ResolveFn,
const InputMatcher &IM, const OutputMatchers &...OMS) {
+ llvm::SmallSet IgnoredDiagnostics{};
auto TU = tooling::buildASTFromCodeWithArgs(Code, {"-std=c++23"
@@ -41,7 +61,16 @@ template
void expectResolution(llvm::StringRef Code, ResolveFnT ResolveFn,
const InputMatcher &IM, const OutputMatchers &...OMS) {
+ llvm::SmallSet IgnoredDiagnostics{};
HighCommander4 wrote:
I would hold off on adding
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/6] Added the mips extension instructions like ehb,ihb and
@@ -658,13 +658,15 @@ class ASTBuilderAction : public ToolAction {
Invocation->getDiagnosticOpts(),
DiagConsumer,
/*ShouldOwnClient=*/false),
-
@@ -238,6 +239,7 @@ std::unique_ptr buildASTFromCodeWithArgs(
ArgumentsAdjuster Adjuster = getClangStripDependencyFileAdjuster(),
const FileContentMappings &VirtualMappedFiles = FileContentMappings(),
DiagnosticConsumer *DiagConsumer = nullptr,
+CaptureDiagsKind
@@ -41,7 +61,16 @@ template
void expectResolution(llvm::StringRef Code, ResolveFnT ResolveFn,
const InputMatcher &IM, const OutputMatchers &...OMS) {
+ llvm::SmallSet IgnoredDiagnostics{};
auto TU = tooling::buildASTFromCodeWithArgs(Code, {"-std=c++23"
mizvekov wrote:
> If they are including `Type.h` in files that only exist downstream that will
> force them to deal with this both times around, unless I am missing something.
No, they would only have had to do that if they had rebased in between this
commit landing and the next one, but that
@@ -0,0 +1,354 @@
+//===- unittest/Format/NumericLiteralCaseTest.cpp
-===//
+//
+// 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
@@ -0,0 +1,354 @@
+//===- unittest/Format/NumericLiteralCaseTest.cpp
-===//
+//
+// 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
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/151590
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,354 @@
+//===- unittest/Format/NumericLiteralCaseTest.cpp
-===//
+//
+// 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
@@ -0,0 +1,176 @@
+//===--- NumericLiteralCaseFixer.cpp *- 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: Ap
@@ -0,0 +1,354 @@
+//===- unittest/Format/NumericLiteralCaseTest.cpp
-===//
+//
+// 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
@@ -0,0 +1,176 @@
+//===--- NumericLiteralCaseFixer.cpp *- 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: Ap
@@ -0,0 +1,176 @@
+//===--- NumericLiteralCaseFixer.cpp *- 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: Ap
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-quick`
running on `linaro-clang-aarch64-quick` while building `clang` at step 5 "ninja
check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/65/builds/21926
Here is the relevant pie
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/5] Added the mips extension instructions like ehb,ihb and
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/4] Added the mips extension instructions like ehb,ihb and
https://github.com/ukalappa-mips updated
https://github.com/llvm/llvm-project/pull/155747
>From 7dca481baec6862b8abed7a912f5666bb6caef17 Mon Sep 17 00:00:00 2001
From: Umesh Kalappa
Date: Thu, 28 Aug 2025 04:53:00 +
Subject: [PATCH 1/3] Added the mips extension instructions like ehb,ihb and
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/155965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov auto_merge_disabled
https://github.com/llvm/llvm-project/pull/155965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov auto_merge_enabled
https://github.com/llvm/llvm-project/pull/155965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Arghnews wrote:
Have made updates, neatened/refactored, now passes all tests with normal and
experimental interpreter, have formatted etc.
@tbaederr @RKSimon please review, thanks!
https://github.com/llvm/llvm-project/pull/155542
___
cfe-commits mail
https://github.com/brandb97 updated
https://github.com/llvm/llvm-project/pull/151428
From 53c90d4f81bf9d10949d2f398a8ccfa0c83af52d Mon Sep 17 00:00:00 2001
From: Lidong Yan
Date: Tue, 26 Aug 2025 19:40:05 +0800
Subject: [PATCH] [clang-format] allow short function body on a single line
When set
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/155965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mizvekov edited
https://github.com/llvm/llvm-project/pull/155965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Matheus Izvekov (mizvekov)
Changes
…context
Fix an error in the logic meant to handle a redeclaration such as:
```C++
struct A {
struct __attribute__((foo)) A *ptr;
};
```
In the declaration of ptr, we must introduce a new redeclaration
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/155965
…context
Fix an error in the logic meant to handle a redeclaration such as:
```C++
struct A {
struct __attribute__((foo)) A *ptr;
};
```
In the declaration of ptr, we must introduce a new redeclaration of A i
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/151428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/151428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/151428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -185,11 +185,18 @@ TEST_F(FormatTestTableGen, SimpleValue6) {
" i32:$dst6,// dst6\n"
" i32:$dst7 // dst7\n"
" );\n"
- " let DAGArgBang = (!c
@@ -185,11 +185,18 @@ TEST_F(FormatTestTableGen, SimpleValue6) {
" i32:$dst6,// dst6\n"
" i32:$dst7 // dst7\n"
" );\n"
- " let DAGArgBang = (!c
@@ -14996,6 +14996,29 @@ TEST_F(FormatTest, SplitEmptyFunctionButNotRecord) {
Style);
}
+TEST_F(FormatTest, SplitShortFunction) {
+ FormatStyle Style = getLLVMStyleWithColumns(40);
brandb97 wrote:
Got it, will fix.
https://github.com/llvm/llv
brandb97 wrote:
owenca ***@***.***> writes:
>
>
> @owenca commented on this pull request.
> In clang/unittests/Format/FormatTest.cpp:
> > +TEST_F(FormatTest, SplitShortFunction) {
> + FormatStyle Style = getLLVMStyleWithColumns(40);
>
> We've been consistently using auto in place of FormatStyl
@@ -14996,6 +14996,29 @@ TEST_F(FormatTest, SplitEmptyFunctionButNotRecord) {
Style);
}
+TEST_F(FormatTest, SplitShortFunction) {
+ FormatStyle Style = getLLVMStyleWithColumns(40);
owenca wrote:
We've been consistently using `auto` in place of
@@ -14996,6 +14996,29 @@ TEST_F(FormatTest, SplitEmptyFunctionButNotRecord) {
Style);
}
+TEST_F(FormatTest, SplitShortFunction) {
+ FormatStyle Style = getLLVMStyleWithColumns(40);
+ Style.AllowShortFunctionsOnASingleLine = FormatStyle::SFS_None;
+ Style.Allo
mpark wrote:
It's just that this is a failing test case that I'd like to ship with a fix.
https://github.com/llvm/llvm-project/pull/155948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commi
@@ -0,0 +1,56 @@
+// -*- 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-
https://github.com/yingcong-wu updated
https://github.com/llvm/llvm-project/pull/154902
>From 57ebc553828bf696b4bde30789e609ba96e94d6d Mon Sep 17 00:00:00 2001
From: Wu Yingcong
Date: Fri, 22 Aug 2025 15:16:47 +0800
Subject: [PATCH 1/8] fix missing by one bug
---
libunwind/src/DwarfParser.hpp
https://github.com/yingcong-wu updated
https://github.com/llvm/llvm-project/pull/154902
>From 57ebc553828bf696b4bde30789e609ba96e94d6d Mon Sep 17 00:00:00 2001
From: Wu Yingcong
Date: Fri, 22 Aug 2025 15:16:47 +0800
Subject: [PATCH 1/7] fix missing by one bug
---
libunwind/src/DwarfParser.hpp
https://github.com/mizvekov auto_merge_enabled
https://github.com/llvm/llvm-project/pull/155959
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Matheus Izvekov (mizvekov)
Changes
The code being removed became non-functional when injected class names were
removed from the redecl chain in 470c454a6176ef31474553e408c90f5ee630df89, but
that commit didn't clean the code base out of al
https://github.com/mizvekov created
https://github.com/llvm/llvm-project/pull/155959
The code being removed became non-functional when injected class names were
removed from the redecl chain in 470c454a6176ef31474553e408c90f5ee630df89, but
that commit didn't clean the code base out of all the
https://github.com/brandb97 edited
https://github.com/llvm/llvm-project/pull/151428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Aiden Grossman (boomanaiden154)
Changes
Currently the clang CMake caches use FE PGO for instrumentation
(LLVM_BUILD_INSTRUMENTED=ON). However, IRPGO is generally regarded as better
for performance. I am measuring about a 1.5% performance
1 - 100 of 549 matches
Mail list logo
