zyn0217 wrote:
@shafik @AaronBallman Here is the complete backstory:
We were not performing well in out-of-line concept comparison, especially for
expressions involving packs. I tried to address these issues around commit
04d20b170, but it quickly became evident that the fix was insufficient f
Author: Djordje Todorovic
Date: 2025-01-28T08:04:09+01:00
New Revision: 0cb7636a462a8d4209e2b6344304eb43f02853eb
URL:
https://github.com/llvm/llvm-project/commit/0cb7636a462a8d4209e2b6344304eb43f02853eb
DIFF:
https://github.com/llvm/llvm-project/commit/0cb7636a462a8d4209e2b6344304eb43f02853eb.d
https://github.com/cor3ntin updated
https://github.com/llvm/llvm-project/pull/124708
>From fb5d3dafcc72e815ed7122dc33b4622a26c16cd8 Mon Sep 17 00:00:00 2001
From: Corentin Jabot
Date: Sat, 25 Jan 2025 16:54:18 +0100
Subject: [PATCH 1/4] [Clang] call HandleImmediateInvocation before checking
fo
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/124575
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
Do we need a changelog here? I'm on the fence but it would not hurt
https://github.com/llvm/llvm-project/pull/124575
___
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: cor3ntin (cor3ntin)
Changes
HandleImmediateInvocation can call MarkExpressionAsImmediateEscalating
and should always be called before CheckImmediateEscalatingFunctionDefinition.
However, we were not doing that in `ActFunctionBody`.
F
https://github.com/philnik777 updated
https://github.com/llvm/llvm-project/pull/119712
>From 0f1a11175bc7c3953e332159fe61192ad29e8c42 Mon Sep 17 00:00:00 2001
From: Nikolas Klauser
Date: Thu, 12 Dec 2024 11:35:49 +0100
Subject: [PATCH] [clang] Extend diagnose_if to accept more detailed warning
https://github.com/djtodoro closed
https://github.com/llvm/llvm-project/pull/121394
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
cor3ntin wrote:
Much better
https://llvm-compile-time-tracker.com/compare_clang.php?from=0e372c3ea31da276ac67c5972e4ef63514577e9c&to=f58459d08ac3eae3b1c77fb4808da60b6e5bb7ec&stat=instructions%3Au&sortBy=interestingness
https://github.com/llvm/llvm-project/pull/124708
___
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 0e372c3ea31da276ac67c5972e4ef63514577e9c
f58459d08ac3eae3b1c77fb4808da60b6e5bb7ec --e
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/124708
HandleImmediateInvocation can call MarkExpressionAsImmediateEscalating
and should always be called before CheckImmediateEscalatingFunctionDefinition.
However, we were not doing that in `ActFunctionBody`.
@@ -1046,6 +1046,20 @@ struct RISCVOperand final : public MCParsedAsmOperand {
isInt<26>(fixImmediateForRV32(Imm, isRV64Imm()));
}
+ bool isImm32() const {
+int64_t Imm;
+RISCVMCExpr::VariantKind VK = RISCVMCExpr::VK_RISCV_None;
+if (!isImm())
+
@@ -245,6 +260,25 @@ class QCIRVInstESStore funct3, bits<2> funct2,
string opcodestr>
(ins GPRMem:$rs2, GPR:$rs1, simm26:$imm),
opcodestr, "$rs2, ${imm}(${rs1})">;
+class QCIRVInstEAI funct3, bits<1> funct1, string opcodestr>
+:
@@ -34,6 +34,21 @@ def uimm11 : RISCVUImmLeafOp<11>;
def simm26 : RISCVSImmLeafOp<26>;
+// 32-bit Immediate, used by RV32 Instructions in 32-bit operations, so no
+// sign-/zero-extension. This is represented internally as a signed 32-bit
value.
+def imm32 : RISCVOp {
+ let
https://github.com/leijurv updated
https://github.com/llvm/llvm-project/pull/118046
>From 1caf823165b16f6701993d586df51d5cdbf0885e Mon Sep 17 00:00:00 2001
From: Leijurv
Date: Fri, 29 Nov 2024 21:54:36 -0600
Subject: [PATCH 01/13] [clang-format] Add BreakBeforeTemplateClose option
---
clang/d
https://github.com/leijurv edited
https://github.com/llvm/llvm-project/pull/118046
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hchandel updated
https://github.com/llvm/llvm-project/pull/124706
>From 81d1b6240cfaedb521d4f0efd54744114577f455 Mon Sep 17 00:00:00 2001
From: Harsh Chandel
Date: Fri, 24 Jan 2025 16:43:12 +0530
Subject: [PATCH 1/3] [RISCV] Add Qualcomm uC Xqcilia (Large Immediate
Arithmeti
https://github.com/ampandey-1995 updated
https://github.com/llvm/llvm-project/pull/123922
>From 5ec4303eb089038db1965b844b8b3bec136207bf Mon Sep 17 00:00:00 2001
From: Amit Pandey
Date: Wed, 22 Jan 2025 15:41:41 +0530
Subject: [PATCH] [Driver][ASan] Refactor Clang-Driver "Sanitizer Bitcode"
li
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/124699
>From 24eab93061adf9a241a0b0b418ea0750e2b66b9d Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Mon, 27 Jan 2025 01:22:31 -0500
Subject: [PATCH] [clang][Sema] Generalize
HeuristicResolver::resolveTypeTo
Shivam7-1 wrote:
> You need to add a test. We have ASan buildbots, but they only run post-merge.
> You can use `LLVM_USE_SANITIZER` [CMake
> option](https://llvm.org/docs/CMake.html#:~:text=llvm_use_sanitizer%3Astring)
> to check locally that Clang crashes on your test without the fix.
Hii @E
@@ -389,11 +390,11 @@ bool findOrdinaryMember(const CXXRecordDecl *RD,
CXXBasePath &Path,
bool HeuristicResolverImpl::findOrdinaryMemberInDependentClasses(
const CXXBaseSpecifier *Specifier, CXXBasePath &Path,
DeclarationName Name) {
- CXXRecordDecl *RD =
- resol
https://github.com/zyn0217 approved this pull request.
One nit, else LGTM
https://github.com/llvm/llvm-project/pull/124699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 edited
https://github.com/llvm/llvm-project/pull/124699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/leijurv edited
https://github.com/llvm/llvm-project/pull/118046
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1523,9 +1570,25 @@ void
Sema::CheckCompleteDecompositionDeclaration(DecompositionDecl *DD) {
// If the type of the decomposition is dependent, then so is the type of
// each binding.
if (DecompType->isDependentType()) {
-for (auto *B : DD->bindings())
- B->s
https://github.com/leijurv updated
https://github.com/llvm/llvm-project/pull/118046
>From 1caf823165b16f6701993d586df51d5cdbf0885e Mon Sep 17 00:00:00 2001
From: Leijurv
Date: Fri, 29 Nov 2024 21:54:36 -0600
Subject: [PATCH 01/13] [clang-format] Add BreakBeforeTemplateClose option
---
clang/d
@@ -11220,6 +11220,333 @@ TEST_F(FormatTest,
WrapsTemplateDeclarationsWithComments) {
Style);
}
+TEST_F(FormatTest, BreakBeforeTemplateCloser) {
+ FormatStyle Style = getGoogleStyle(FormatStyle::LK_Cpp);
+ // Begin with tests covering the case where there is no constr
@@ -3423,6 +3425,13 @@ VarDecl *BindingDecl::getHoldingVar() const {
return VD;
}
+llvm::ArrayRef BindingDecl::getBindingPackExprs() const {
+ if (!Binding)
+return {};
ricejasonf wrote:
Added. The assert was being triggered in one spot that I realized
@@ -11220,6 +11220,333 @@ TEST_F(FormatTest,
WrapsTemplateDeclarationsWithComments) {
Style);
}
+TEST_F(FormatTest, BreakBeforeTemplateCloser) {
+ FormatStyle Style = getGoogleStyle(FormatStyle::LK_Cpp);
+ // Begin with tests covering the case where there is no constr
https://github.com/leijurv updated
https://github.com/llvm/llvm-project/pull/118046
>From 1caf823165b16f6701993d586df51d5cdbf0885e Mon Sep 17 00:00:00 2001
From: Leijurv
Date: Fri, 29 Nov 2024 21:54:36 -0600
Subject: [PATCH 01/13] [clang-format] Add BreakBeforeTemplateClose option
---
clang/d
@@ -0,0 +1,16 @@
+// RUN: %clang %s -S -emit-llvm -fprofile-generate -fprofile-continuous -o - |
FileCheck %s --check-prefix=IRPGO
w2yehia wrote:
In this test, I'm testing both the driver (lines 4,5,8) and the clang codegen
(lines 1,2,3,6).
Should I split it in
dgg5503 wrote:
Hi @steakhal ,
Thanks for providing such detailed answers to my questions! Given your
response, I feel like the design of my implementation is on the right track,
although feel free to critique it further when you have the chance. One thing I
do need to look further into is app
https://github.com/zyn0217 approved this pull request.
https://github.com/llvm/llvm-project/pull/124675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/adam-yang updated
https://github.com/llvm/llvm-project/pull/123428
>From cedd93faaa75c343fda7a21ecf813249cf08609a Mon Sep 17 00:00:00 2001
From: Adam Yang <31109344+adam-y...@users.noreply.github.com>
Date: Fri, 17 Jan 2025 16:14:44 -0800
Subject: [PATCH 1/6] Added WaveActiveM
@@ -11682,6 +11683,7 @@ class Sema final : public SemaBase {
SmallVectorImpl &SugaredConverted,
SmallVectorImpl &CanonicalConverted,
CheckTemplateArgumentKind CTAK, bool PartialOrdering,
+
thurstond wrote:
> Lets move clang part into a new PR?
I've removed the clang part. Separate PR coming soon (tm).
https://github.com/llvm/llvm-project/pull/124211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/m
@@ -2314,7 +2314,6 @@ bool CompilerInvocation::ParseCodeGenArgs(CodeGenOptions
&Opts, ArgList &Args,
Opts.SanitizeSkipHotCutoffs = parseSanitizerWeightedKinds(
"-fsanitize-skip-hot-cutoff=",
Args.getAllArgValues(OPT_fsanitize_skip_hot_cutoff_EQ), Diags);
-
--
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/124211
>From 5d1944f522152a7356d8e8147d3b3c5939c32526 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Wed, 22 Jan 2025 18:59:07 +
Subject: [PATCH 1/6] [ubsan] Plumb through -fsanitize-skip-hot-cutoff to
Lower
@@ -2314,7 +2314,6 @@ bool CompilerInvocation::ParseCodeGenArgs(CodeGenOptions
&Opts, ArgList &Args,
Opts.SanitizeSkipHotCutoffs = parseSanitizerWeightedKinds(
"-fsanitize-skip-hot-cutoff=",
Args.getAllArgValues(OPT_fsanitize_skip_hot_cutoff_EQ), Diags);
-
--
@@ -71,7 +72,8 @@ static void emitRemark(IntrinsicInst *II,
OptimizationRemarkEmitter &ORE,
static bool removeUbsanTraps(Function &F, const BlockFrequencyInfo &BFI,
const ProfileSummaryInfo *PSI,
- OptimizationRemarkEmi
AlexVlx wrote:
> > It is six months old, but it is relatively difficult to keep track of all
> > the concurrent non-communicating swimlanes - I do apologise for not having
> > done so though! It is possible that we are microfocusing on a potential use
> > case, but there are other valid ones.
https://github.com/vitalybuka approved this pull request.
https://github.com/llvm/llvm-project/pull/124211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -828,11 +828,66 @@ parseLowerAllowCheckPassOptions(StringRef Params) {
StringRef ParamName;
std::tie(ParamName, Params) = Params.split(';');
-return make_error(
-formatv("invalid LowerAllowCheck pass parameter '{0}' ", ParamName)
-.str(),
-
@@ -2,6 +2,14 @@
; RUN: opt < %s -passes='function(lower-allow-check)' -S | FileCheck %s
--check-prefixes=NOPROFILE
; RUN: opt < %s -passes='function(lower-allow-check)'
-lower-allow-check-random-rate=0 -S | FileCheck %s --check-prefixes=NONE
; RUN: opt < %s -passes='function
jhuber6 wrote:
> It is six months old, but it is relatively difficult to keep track of all the
> concurrent non-communicating swimlanes - I do apologise for not having done
> so though! It is possible that we are microfocusing on a potential use case,
> but there are other valid ones. For exam
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/124211
>From 5d1944f522152a7356d8e8147d3b3c5939c32526 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Wed, 22 Jan 2025 18:59:07 +
Subject: [PATCH 1/5] [ubsan] Plumb through -fsanitize-skip-hot-cutoff to
Lower
Author: Nathan Ridge
Date: 2025-01-27T19:51:30-05:00
New Revision: 194c74e0166876ef2c5e0ff5e98a4b5a5442138c
URL:
https://github.com/llvm/llvm-project/commit/194c74e0166876ef2c5e0ff5e98a4b5a5442138c
DIFF:
https://github.com/llvm/llvm-project/commit/194c74e0166876ef2c5e0ff5e98a4b5a5442138c.diff
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/124211
>From 5d1944f522152a7356d8e8147d3b3c5939c32526 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Wed, 22 Jan 2025 18:59:07 +
Subject: [PATCH 1/7] [ubsan] Plumb through -fsanitize-skip-hot-cutoff to
Lower
@@ -828,11 +828,66 @@ parseLowerAllowCheckPassOptions(StringRef Params) {
StringRef ParamName;
std::tie(ParamName, Params) = Params.split(';');
-return make_error(
-formatv("invalid LowerAllowCheck pass parameter '{0}' ", ParamName)
-.str(),
-
https://github.com/andykaylor updated
https://github.com/llvm/llvm-project/pull/124650
>From 41546e2d096310d8a1474539b5036152c7df3d11 Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Mon, 27 Jan 2025 14:54:10 -0800
Subject: [PATCH 1/2] [CIR] Add framework for CIR to LLVM IR lowering
Create the
@@ -2,6 +2,14 @@
; RUN: opt < %s -passes='function(lower-allow-check)' -S | FileCheck %s
--check-prefixes=NOPROFILE
; RUN: opt < %s -passes='function(lower-allow-check)'
-lower-allow-check-random-rate=0 -S | FileCheck %s --check-prefixes=NONE
; RUN: opt < %s -passes='function
@@ -107,7 +108,18 @@ static bool removeUbsanTraps(Function &F, const
BlockFrequencyInfo &BFI,
case Intrinsic::allow_runtime_check: {
++NumChecksTotal;
-bool ToRemove = ShouldRemove(BB);
+bool ToRemove = ShouldRemoveRandom();
+
+unsigned i
@@ -107,7 +108,18 @@ static bool removeUbsanTraps(Function &F, const
BlockFrequencyInfo &BFI,
case Intrinsic::allow_runtime_check: {
++NumChecksTotal;
-bool ToRemove = ShouldRemove(BB);
+bool ToRemove = ShouldRemoveRandom();
+
+unsigned i
@@ -81,21 +83,20 @@ static bool removeUbsanTraps(Function &F, const
BlockFrequencyInfo &BFI,
return *Rng;
};
- auto ShouldRemoveHot = [&](const BasicBlock &BB) {
-return HotPercentileCutoff.getNumOccurrences() && PSI &&
- PSI->isHotCountNthPercentile(
-
@@ -81,21 +83,20 @@ static bool removeUbsanTraps(Function &F, const
BlockFrequencyInfo &BFI,
return *Rng;
};
- auto ShouldRemoveHot = [&](const BasicBlock &BB) {
-return HotPercentileCutoff.getNumOccurrences() && PSI &&
- PSI->isHotCountNthPercentile(
-
@@ -71,7 +72,8 @@ static void emitRemark(IntrinsicInst *II,
OptimizationRemarkEmitter &ORE,
static bool removeUbsanTraps(Function &F, const BlockFrequencyInfo &BFI,
const ProfileSummaryInfo *PSI,
- OptimizationRemarkEmi
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 ec15b242505a46ea7d195a6520fb869a80a2cd10
7b6b0166835faff53665321342b47b0887ee99a6 --e
@@ -108,14 +108,17 @@ static Value *getBoundsCheckCond(Value *Ptr, Value
*InstVal,
return Or;
}
-static CallInst *InsertTrap(BuilderTy &IRB, bool DebugTrapBB) {
+static CallInst *InsertTrap(BuilderTy &IRB, bool DebugTrapBB,
thurstond wrote:
Done (https://g
https://github.com/thurstond edited
https://github.com/llvm/llvm-project/pull/124211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HighCommander4 closed
https://github.com/llvm/llvm-project/pull/124628
___
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
`openmp-offload-libc-amdgpu-runtime` running on `omp-vega20-1` while building
`clang` at step 7 "Add check check-offload".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/73/builds/12525
Here is the r
https://github.com/thurstond edited
https://github.com/llvm/llvm-project/pull/124211
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thurstond updated
https://github.com/llvm/llvm-project/pull/124211
>From 5d1944f522152a7356d8e8147d3b3c5939c32526 Mon Sep 17 00:00:00 2001
From: Thurston Dang
Date: Wed, 22 Jan 2025 18:59:07 +
Subject: [PATCH 1/4] [ubsan] Plumb through -fsanitize-skip-hot-cutoff to
Lower
https://github.com/Un1q32 updated
https://github.com/llvm/llvm-project/pull/124651
>From d4e97c4113086c3d2dfa3bf6e9ecfee377f8c4b6 Mon Sep 17 00:00:00 2001
From: Un1q32
Date: Mon, 27 Jan 2025 18:00:34 -0500
Subject: [PATCH 1/2] [Clang] Link libgcc_s.1.dylib when building for macOS
10.5 and olde
https://github.com/antangelo closed
https://github.com/llvm/llvm-project/pull/123875
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: antangelo
Date: 2025-01-27T18:59:12-05:00
New Revision: f94c481543bdd3b11a668ad78d46593cf974788f
URL:
https://github.com/llvm/llvm-project/commit/f94c481543bdd3b11a668ad78d46593cf974788f
DIFF:
https://github.com/llvm/llvm-project/commit/f94c481543bdd3b11a668ad78d46593cf974788f.diff
LOG
https://github.com/w2yehia edited
https://github.com/llvm/llvm-project/pull/124353
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,41 @@
+//- LowerToLLVM.cpp - Lowering from CIR to LLVMIR
-===//
+//
+// 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
https://github.com/bcardosolopes approved this pull request.
Seems like there are some CI failures (format related?) but LGTM once those are
fixed
https://github.com/llvm/llvm-project/pull/124650
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/dwblaikie approved this pull request.
Looks great - thanks!
https://github.com/llvm/llvm-project/pull/124643
___
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: None (Un1q32)
Changes
There are a lot of symbols that the compiler expects to be in libSystem that
were only in libgcc_s before 10.6, like libunwind things and some builtins on
i386. We already do this for iOS versions < 5.0 for the same
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: None (Un1q32)
Changes
There are a lot of symbols that the compiler expects to be in libSystem that
were only in libgcc_s before 10.6, like libunwind things and some builtins on
i386. We already do this for iOS versions < 5.0 for t
https://github.com/Un1q32 created
https://github.com/llvm/llvm-project/pull/124651
There are a lot of symbols that the compiler expects to be in libSystem that
were only in libgcc_s before 10.6, like libunwind things and some builtins on
i386. We already do this for iOS versions < 5.0 for the
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 3861b9db882d5637725ceeccb801c2bb837e8fc5
41546e2d096310d8a1474539b5036152c7df3d11 --e
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Andy Kaylor (andykaylor)
Changes
Create the skeleton framework for lowering from ClangIR to LLVM IR. This
initial implementation just creates an empty LLVM IR module. Actual lowering of
even minimal ClangIR is deferred to a later patch.
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/124650
Create the skeleton framework for lowering from ClangIR to LLVM IR. This
initial implementation just creates an empty LLVM IR module. Actual lowering of
even minimal ClangIR is deferred to a later patch.
>F
https://github.com/rnk approved this pull request.
https://github.com/llvm/llvm-project/pull/123548
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -51,28 +57,71 @@ class StringToOffsetTable {
return II->second;
}
- // Emit the string using string literal concatenation, for better readability
- // and searchability.
- void EmitStringLiteralDef(raw_ostream &OS, const Twine &Decl,
-co
Author: cor3ntin
Date: 2025-01-27T23:51:19+01:00
New Revision: 0e372c3ea31da276ac67c5972e4ef63514577e9c
URL:
https://github.com/llvm/llvm-project/commit/0e372c3ea31da276ac67c5972e4ef63514577e9c
DIFF:
https://github.com/llvm/llvm-project/commit/0e372c3ea31da276ac67c5972e4ef63514577e9c.diff
LOG:
cor3ntin wrote:
> FYI this one seems like a fairly substantial compile-time regression for C++
> code:
> https://llvm-compile-time-tracker.com/compare_clang.php?from=ad9da92cf6f735747ef04fd56937e1d76819e503&to=5815a311050ae218ebcda53adeee24ed96851943&stat=instructions%3Au&sortBy=interestingness
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: cor3ntin (cor3ntin)
Changes
Reverts llvm/llvm-project#124414
Turns out to be an important compile time regression, I'll come up with a less
disruptive approach
---
Full diff: https://github.com/llvm/llvm-project/pull/124646.diff
6 File
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/124646
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin created
https://github.com/llvm/llvm-project/pull/124646
Reverts llvm/llvm-project#124414
Turns out to be an important compile time regression, I'll come up with a less
disruptive approach
>From 6660401456710db91998bbfe60f070a8f05edabe Mon Sep 17 00:00:00 2001
From
nikic wrote:
FYI this one seems like a fairly substantial compile-time regression for C++
code:
https://llvm-compile-time-tracker.com/compare_clang.php?from=ad9da92cf6f735747ef04fd56937e1d76819e503&to=5815a311050ae218ebcda53adeee24ed96851943&stat=instructions%3Au&sortBy=interestingness
https:/
https://github.com/Artem-B approved this pull request.
https://github.com/llvm/llvm-project/pull/124626
___
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-format
Author: Gedare Bloom (gedare)
Changes
An attribute in a class/struct declaration confuses the parser to treat the
identifier following the attribute as a function or variable name.
Fixes #124574
---
Full diff: https://github.com/llvm/llvm
https://github.com/gedare created
https://github.com/llvm/llvm-project/pull/124634
An attribute in a class/struct declaration confuses the parser to treat the
identifier following the attribute as a function or variable name.
Fixes #124574
>From db6e09dca9d39b1c21543a069e7a1d1bf3997c0c Mon Se
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier
Message-ID:
In-Reply-To:
@@ -10134,6 +10137,24 @@ def note_format_fix_specifier : Note<"did you mean to
use '%0'?">;
def note_printf_c_str: Note<"did you mean to
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier
Message-ID:
In-Reply-To:
@@ -10134,6 +10137,24 @@ def note_format_fix_specifier : Note<"did you mean to
use '%0'?">;
def note_printf_c_str: Note<"did you mean to
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier
Message-ID:
In-Reply-To:
@@ -10134,6 +10137,24 @@ def note_format_fix_specifier : Note<"did you mean to
use '%0'?">;
def note_printf_c_str: Note<"did you mean to
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier
Message-ID:
In-Reply-To:
https://github.com/shafik edited
https://github.com/llvm/llvm-project/pull/116708
___
cfe-commits mailing list
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier ,
=?utf-8?q?Félix?= Cloutier
Message-ID:
In-Reply-To:
https://github.com/shafik commented:
Apologies if I somehow missed it but it looks like we are not really covering
the diagnostics fully and we need to in
https://github.com/ilovepi closed
https://github.com/llvm/llvm-project/pull/121820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Paul Kirth
Date: 2025-01-27T13:44:18-08:00
New Revision: 6578790076fed5467bf613e48f7a0ad2f50f041a
URL:
https://github.com/llvm/llvm-project/commit/6578790076fed5467bf613e48f7a0ad2f50f041a
DIFF:
https://github.com/llvm/llvm-project/commit/6578790076fed5467bf613e48f7a0ad2f50f041a.diff
LO
https://github.com/zeroomega approved this pull request.
https://github.com/llvm/llvm-project/pull/121820
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/efriedma-quic approved this pull request.
LGTM
In theory, handling streaming mode doesn't seem too challenging here: you mark
the outlined functions locally_streaming, and you're done, I think. That said,
I'm not sure how you define the semantics for ZA, so maybe better to
=?utf-8?q?André?= Brand ,
=?utf-8?q?André?= Brand
Message-ID:
In-Reply-To:
shafik wrote:
> @cor3ntin Thanks! I just added some details to the GitHub description. I hope
> it's not too verbose. And I rebased the branch to resolve the conflict in the
> release notes.
Terse summaries are reall
@@ -785,6 +786,34 @@ static void addPGOAndCoverageFlags(const ToolChain &TC,
Compilation &C,
D.Diag(diag::err_drv_unsupported_option_argument)
<< A->getSpelling() << Val;
}
+ if (const auto *A = Args.getLastArg(options::OPT_fprofile_continuous)) {
+if (!
efriedma-quic wrote:
Looking a bit more, I'm not sure I buy the premise that we don't need to mangle
lambdas written in unevaluated contexts. Consider the following related
example:
```
template
using simd_vector = Tp;
template
using simd_vector_underlying_type_t
= decltype([](simd_vecto
@@ -133,6 +133,16 @@ std::string getDefaultProfileGenName() {
: "default_%m.profraw";
}
+// Path and name of file used for profile generation
+std::string getProfileGenName(const CodeGenOptions &CodeGenOpts) {
w2yehia wrote:
sure. FYI I was follo
1 - 100 of 597 matches
Mail list logo
