https://github.com/alexrp edited
https://github.com/llvm/llvm-project/pull/119997
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/amane-ame updated
https://github.com/llvm/llvm-project/pull/119428
From 3a4c1a924faef3a7a09126694fcb943bd7083451 Mon Sep 17 00:00:00 2001
From: amane-ame
Date: Wed, 11 Dec 2024 02:13:43 +0800
Subject: [PATCH 1/2] Fix crashes when the macro expansion is empty
---
clang/lib/F
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/119989
>From fa8d1b12eee0164f2b4c8223281d0e59dfa693e1 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Sat, 14 Dec 2024 15:25:44 -0700
Subject: [PATCH 1/2] [clang-format] detect nesting in template strings
The helper
Author: Owen Pan
Date: 2024-12-14T19:42:43-08:00
New Revision: 8c681a929b8684f5a4ad2ebd4e3e4f20036a9595
URL:
https://github.com/llvm/llvm-project/commit/8c681a929b8684f5a4ad2ebd4e3e4f20036a9595
DIFF:
https://github.com/llvm/llvm-project/commit/8c681a929b8684f5a4ad2ebd4e3e4f20036a9595.diff
LOG:
https://github.com/alexrp created
https://github.com/llvm/llvm-project/pull/119997
LLVM's WebAssembly backend supports the `noredzone` function attribute, but
this support was not exposed in Clang.
Note: I don't have commit access.
From 28834f7a42511905d8c1e00cb16ea7cdc70bf0c1 Mon Sep 17 00:0
llvmbot wrote:
@llvm/pr-subscribers-clang-driver
Author: Alex Rønne Petersen (alexrp)
Changes
LLVM's WebAssembly backend supports the `noredzone` function attribute, but
this support was not exposed in Clang.
Note: I don't have commit access.
---
Full diff: https://github.com/llvm/llvm-
https://github.com/HighCommander4 approved this pull request.
LGTM (I made a small tweak to the test name)
https://github.com/llvm/llvm-project/pull/119735
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/l
HighCommander4 wrote:
Hmm, the test is failing for me locally. Is it passing for you?
(I'm not quite sure from the Buildkite logs whether it ran this test or not.)
https://github.com/llvm/llvm-project/pull/119735
___
cfe-commits mailing list
cfe-commi
https://github.com/HighCommander4 updated
https://github.com/llvm/llvm-project/pull/119735
>From 04757e7d94ce5db11bb397accb0b1c0523d351ba Mon Sep 17 00:00:00 2001
From: Joseph Huber
Date: Thu, 12 Dec 2024 12:15:32 -0600
Subject: [PATCH 1/3] [clangd] Index reserved symbols from `*intrin.h` syste
https://github.com/alexrp updated
https://github.com/llvm/llvm-project/pull/116608
From 3a35fd5823545560041b4e091519895b7f7c89bb Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alex=20R=C3=B8nne=20Petersen?=
Date: Mon, 18 Nov 2024 13:12:30 +0100
Subject: [PATCH] [libunwind] Fix compilation for the x32
https://github.com/alexrp updated
https://github.com/llvm/llvm-project/pull/116608
From 9955135c2027d268b9260adabaaf021848f91059 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Alex=20R=C3=B8nne=20Petersen?=
Date: Mon, 18 Nov 2024 13:12:30 +0100
Subject: [PATCH] [libunwind] Fix compilation for the x32
@@ -0,0 +1,104 @@
+//===--- UseSpanFirstLastCheck.cpp - 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: Ap
Author: Pavel Kosov
Date: 2024-12-14T14:46:47+03:00
New Revision: 1911919682c863643787b30286bb67359c7932f4
URL:
https://github.com/llvm/llvm-project/commit/1911919682c863643787b30286bb67359c7932f4
DIFF:
https://github.com/llvm/llvm-project/commit/1911919682c863643787b30286bb67359c7932f4.diff
L
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/116709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/116709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/philnik777 edited
https://github.com/llvm/llvm-project/pull/116709
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
QrczakMK wrote:
It would be great to see progress in this. Lack of detection of trivially
relocatable types costs extra memory allocations in my project, and I feel
uneasy carving a manual exception for `std::unique_ptr`.
https://github.com/llvm/llvm-project/pull/84621
https://github.com/hjanuschka updated
https://github.com/llvm/llvm-project/pull/118074
>From cb748c34d35b8c0c9ca93a67b111dcf5d7665b34 Mon Sep 17 00:00:00 2001
From: Helmut Januschka
Date: Fri, 29 Nov 2024 10:17:49 +0100
Subject: [PATCH 01/31] [clang-tidy] Add modernize-use-span-first-last check
https://github.com/hjanuschka commented:
ready for re-review
https://github.com/llvm/llvm-project/pull/118074
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kpdev wrote:
@phuang @nico Reverted. I will reland it after investigating the issue with GN
build bot
https://github.com/llvm/llvm-project/pull/118192
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/list
https://github.com/aaronpuchert closed
https://github.com/llvm/llvm-project/pull/116487
___
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-tools-extra
Author: Barnabás Pőcze (pobrn)
Changes
Static member functions can be considered the same way as free functions are,
so do that.
---
1. Not sure how many more tests I should add, since this uses the same code
paths as free functions
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
https://github.com/pobrn created
https://github.com/llvm/llvm-project/pull/119974
Static member functions can be considered the same way as free functions are,
so do that.
---
1. Not sure how many more tests I should add, since this uses the same code
paths as free functions.
2. This is the
Author: Aaron Puchert
Date: 2024-12-14T16:54:43+01:00
New Revision: a999ab44be8994d39d2469c1b4d025c4e1131197
URL:
https://github.com/llvm/llvm-project/commit/a999ab44be8994d39d2469c1b4d025c4e1131197
DIFF:
https://github.com/llvm/llvm-project/commit/a999ab44be8994d39d2469c1b4d025c4e1131197.diff
https://github.com/pobrn updated
https://github.com/llvm/llvm-project/pull/119974
From 2d72c484291862fd8c9d8cea1a3ebadbe37343be Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Barnab=C3=A1s=20P=C5=91cze?=
Date: Sat, 14 Dec 2024 16:57:57 +0100
Subject: [PATCH] [clang-tidy] performance-unnecessary-copy-
https://github.com/kazutakahirata closed
https://github.com/llvm/llvm-project/pull/119949
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Kazu Hirata
Date: 2024-12-14T09:17:56-08:00
New Revision: c35108e24488af1db1914ec083439189e6a7fce6
URL:
https://github.com/llvm/llvm-project/commit/c35108e24488af1db1914ec083439189e6a7fce6
DIFF:
https://github.com/llvm/llvm-project/commit/c35108e24488af1db1914ec083439189e6a7fce6.diff
L
yuxuanchen1997 wrote:
> @SahilPatidar, could you help us with this?
Can we temporarily disable this test when @SahilPatidar is looking for a
solution?
https://github.com/llvm/llvm-project/pull/117475
___
cfe-commits mailing list
cfe-commits@lists.llv
https://github.com/yuxuanchen1997 created
https://github.com/llvm/llvm-project/pull/119978
See discussion on
https://github.com/llvm/llvm-project/pull/117475#issuecomment-2543223954
This test currently invokes the system ld, which can cause issues on some CI
systems where system ld only works
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Yuxuan Chen (yuxuanchen1997)
Changes
See discussion on
https://github.com/llvm/llvm-project/pull/117475#issuecomment-2543223954
This test currently invokes the system ld, which can cause issues on some CI
systems where system ld only wor
Author: Davide Italiano
Date: 2024-12-14T18:25:02Z
New Revision: 9ddcaed3a64c2a187a0cfff4ba8f989c665ae1e5
URL:
https://github.com/llvm/llvm-project/commit/9ddcaed3a64c2a187a0cfff4ba8f989c665ae1e5
DIFF:
https://github.com/llvm/llvm-project/commit/9ddcaed3a64c2a187a0cfff4ba8f989c665ae1e5.diff
LO
Author: Davide Italiano
Date: 2024-12-14T18:26:32Z
New Revision: 61ab36a3e226df32855286dd31a2c3859800475d
URL:
https://github.com/llvm/llvm-project/commit/61ab36a3e226df32855286dd31a2c3859800475d
DIFF:
https://github.com/llvm/llvm-project/commit/61ab36a3e226df32855286dd31a2c3859800475d.diff
LO
dcci wrote:
Given the fix forward didn't work, I reverted this to unblock
```
commit 61ab36a3e226df32855286dd31a2c3859800475d (HEAD -> main, upstream/main)
Author: Davide Italiano
Date: Sat Dec 14 18:25:50 2024 +
Revert "[Clang-REPL] Fix crash during `__run_exit_handlers` with dynami
https://github.com/vgvassilev approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/119978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mgorny wrote:
This test is failing on x86: #119979
https://github.com/llvm/llvm-project/pull/119719
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
SherAndrei wrote:
> Can you expand on your concern about the clash?
The clash is happening in `NodesMap`. See,
1. the matcher which is associated with `rewriteDescendants` gets "Tag0",
2. tags for matchers from `applyfirst` are shifted, they get "Tag1", ...
3. that is why `transformer::detail::
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/119958
Fixes #59066.
>From 393e153c8ce6ec713ad9653f5f37eacc72b17b39 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Sat, 14 Dec 2024 00:33:35 -0800
Subject: [PATCH] [clang-format] Fix a bug in annotating arrows after ini
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Owen Pan (owenca)
Changes
Fixes #59066.
---
Full diff: https://github.com/llvm/llvm-project/pull/119958.diff
3 Files Affected:
- (modified) clang/lib/Format/TokenAnnotator.cpp (+2-1)
- (modified) clang/lib/Format/UnwrappedLinePa
https://github.com/amane-ame updated
https://github.com/llvm/llvm-project/pull/119563
From 659eda3ec76b63418f8b621b004728d9d7bf26ad Mon Sep 17 00:00:00 2001
From: amane-ame
Date: Wed, 11 Dec 2024 22:17:51 +0800
Subject: [PATCH 1/8] [clang] Fix crashes when passing VLA to va_arg
---
clang/lib/
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/119949
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
carlocab wrote:
Obsoleted by 61ab36a3e226df32855286dd31a2c3859800475d
https://github.com/llvm/llvm-project/pull/119978
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/carlosgalvezp approved this pull request.
LGTM, thanks!
https://github.com/llvm/llvm-project/pull/119948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gedare wrote:
> I don't think this is the way to go, or at least not with this name.
>
> How about going the `Custom` way?
Adding a `Custom` option to `AlignAfterOpenBracket` is an interesting proposal.
There have been some ideas floated in other Issues and PRs related to this, for
example:
*
Author: Richard Dzenis
Date: 2024-12-15T00:18:09+02:00
New Revision: 99354f968f64659cbad5c82b0301d851ae54f057
URL:
https://github.com/llvm/llvm-project/commit/99354f968f64659cbad5c82b0301d851ae54f057
DIFF:
https://github.com/llvm/llvm-project/commit/99354f968f64659cbad5c82b0301d851ae54f057.diff
https://github.com/RIscRIpt closed
https://github.com/llvm/llvm-project/pull/119986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -811,10 +816,11 @@ void
ContinuationIndenter::addTokenOnCurrentLine(LineState &State, bool DryRun,
if (!Tok.Previous)
return true;
if (Tok.Previous->isIf())
- return Style.AlignAfterOpenBracket == FormatStyle::BAS_AlwaysBreak;
-return !Tok.Previous->i
@@ -1452,6 +1476,7 @@ FormatStyle getLLVMStyle(FormatStyle::LanguageKind
Language) {
FormatStyle LLVMStyle;
LLVMStyle.AccessModifierOffset = -2;
LLVMStyle.AlignAfterOpenBracket = FormatStyle::BAS_Align;
+ LLVMStyle.AlignAfterOpenBracketBreak = {};
gedar
Author: Congcong Cai
Date: 2024-12-15T05:04:32+08:00
New Revision: aaadaee7b228d7010ff7076f5002ebb96b5e03dc
URL:
https://github.com/llvm/llvm-project/commit/aaadaee7b228d7010ff7076f5002ebb96b5e03dc
DIFF:
https://github.com/llvm/llvm-project/commit/aaadaee7b228d7010ff7076f5002ebb96b5e03dc.diff
https://github.com/HerrCai0907 closed
https://github.com/llvm/llvm-project/pull/119948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vgvassilev wrote:
> Obsoleted by
> [61ab36a](https://github.com/llvm/llvm-project/commit/61ab36a3e226df32855286dd31a2c3859800475d)
That was the right fix, not the revert...
https://github.com/llvm/llvm-project/pull/119978
___
cfe-commits mailing list
https://github.com/gedare created
https://github.com/llvm/llvm-project/pull/119989
The helper to check if a token is in a template string scans too far backward.
It should stop if a different scope is found.
Fixes #107571
>From fa8d1b12eee0164f2b4c8223281d0e59dfa693e1 Mon Sep 17 00:00:00 2001
llvmbot wrote:
@llvm/pr-subscribers-clang-format
Author: Gedare Bloom (gedare)
Changes
The helper to check if a token is in a template string scans too far backward.
It should stop if a different scope is found.
Fixes #107571
---
Full diff: https://github.com/llvm/llvm-project/pull/1199
https://github.com/gedare updated
https://github.com/llvm/llvm-project/pull/112482
>From d625f811a615155b15a007ec3afc9874c52d06c4 Mon Sep 17 00:00:00 2001
From: Gedare Bloom
Date: Tue, 15 Oct 2024 23:55:49 -0600
Subject: [PATCH 1/3] [clang-format] add BinPackLongBracedLists style option
The us
@@ -1208,6 +1208,21 @@ struct FormatStyle {
/// \version 3.7
bool BinPackArguments;
+ /// If ``BinPackArguments`` is ``false`` this option can override it if
+ /// ``true`` when 20 or more items are in a braced initializer list.
gedare wrote:
I rewrote
gedare wrote:
> I think it should be merges with `BinPackArguments` to get an enum
>
> * Never
> * TwentyOrAbove (name is debatable)
> * Always
> * (Leave?)
Currently the 20 item limit is only enforced on C++ initializer lists. Making
it part of the `BinPackArguments` means it has to apply to
https://github.com/smallp-o-p updated
https://github.com/llvm/llvm-project/pull/118186
>From 3b7cf6e65bdfedf8d15e393c9c2f819c4ed70386 Mon Sep 17 00:00:00 2001
From: William Tran-Viet
Date: Sat, 30 Nov 2024 15:53:32 -0500
Subject: [PATCH 1/4] Fix double-quotes in diagnostic when attempting to ac
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-ppc64-aix` running
on `aix-ppc64` while building `clang,llvm` at step 3 "clean-build-dir".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/64/builds/1692
Here is the relevant piece of the build
https://github.com/5chmidti approved this pull request.
Thanks
https://github.com/llvm/llvm-project/pull/119948
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/smallp-o-p updated
https://github.com/llvm/llvm-project/pull/118186
>From 3b7cf6e65bdfedf8d15e393c9c2f819c4ed70386 Mon Sep 17 00:00:00 2001
From: William Tran-Viet
Date: Sat, 30 Nov 2024 15:53:32 -0500
Subject: [PATCH 1/4] Fix double-quotes in diagnostic when attempting to ac
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/119894
>From 6cd26753f380d9ee89d85139a5dc58bc0e4b0632 Mon Sep 17 00:00:00 2001
From: Vitaly Buka
Date: Fri, 13 Dec 2024 08:04:56 -0800
Subject: [PATCH 1/2] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?
5chmidti wrote:
@HerrCai0907 Could you retroactively add a release note? While this hopefully
doesn't exist in anyone's config, but if someone were to look, they'd see why
this was done.
https://github.com/llvm/llvm-project/pull/119948
___
cfe-commi
yuxuanchen1997 wrote:
May I understand what's the intention of this? I would prefer that the original
author figures out a solution that doesn't depend on system linker, which is
what I would consider a proper fix.
https://github.com/llvm/llvm-project/pull/119978
__
https://github.com/yuxuanchen1997 closed
https://github.com/llvm/llvm-project/pull/119978
___
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: Richard Dzenis (RIscRIpt)
Changes
Fix test failure from #119719
84b0f0145887bbfe49fd4dc85490b14108a72cee
Closes #119979
---
Full diff: https://github.com/llvm/llvm-project/pull/119986.diff
1 Files Affected:
- (modified) clang/test/Sema
carlocab wrote:
> > Obsoleted by
> > [61ab36a](https://github.com/llvm/llvm-project/commit/61ab36a3e226df32855286dd31a2c3859800475d)
>
>
>
> That was the right fix, not the revert...
I'm inclined to agree. Could revert the revert then apply this.
https://github.com/llvm/llvm-project/pull/11
https://github.com/RIscRIpt created
https://github.com/llvm/llvm-project/pull/119986
Fix test failure from #119719
84b0f0145887bbfe49fd4dc85490b14108a72cee
Closes #119979
>From 876efed487603b450c56cd4c282449f5bfef52ad Mon Sep 17 00:00:00 2001
From: Richard Dzenis
Date: Sat, 14 Dec 2024 23:32:
RIscRIpt wrote:
I reproduced the problem locally by adding `-m32` flag to `%clang_cl`. Changes
in this PR fix the test failure.
https://github.com/llvm/llvm-project/pull/119986
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
yuxuanchen1997 wrote:
> I still feel it would be simpler to lookup the function for `await_suspend`
> in the awaiter and look for its return type. Then we don't need to care about
> the type of the expression.
After reading a bit, I think it's possible to do this by duplicating a little
bit o
https://github.com/owenca edited
https://github.com/llvm/llvm-project/pull/119989
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2157,6 +2157,13 @@ TEST_F(FormatTestJS, TemplateStringMultiLineExpression) {
" : a,\n"
" : b,\n"
"})}`;");
+
+ verifyFormat("`${\n"
+
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/119989
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AdUhTkJm updated
https://github.com/llvm/llvm-project/pull/119098
>From 56670608becf2032867405778fa7b2b1a36fb3cf Mon Sep 17 00:00:00 2001
From: AdUhTkJm <2292398...@qq.com>
Date: Sun, 8 Dec 2024 08:07:59 +0800
Subject: [PATCH] [Clang] Fix crash for incompatible types in inline
owenca wrote:
Please add unit tests.
https://github.com/llvm/llvm-project/pull/119428
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
74 matches
Mail list logo
