https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114661
>From f52c645a8b0a3664c2520e70b46e248c984037ae Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
SixWeining wrote:
> > Is it the time to remove the FIXME in those tests?
>
> There are several random inconsistencies and I separated the fixes into
> multiple PRs as suggested by Xuerui. The FIXME is removed in #114513.
Oh, I missed those PRs.
https://github.com/llvm/llvm-project/pull/114511
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114660
>From a8caf223a55b16fc4e0aaa483c4af62306137179 Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114660
>From 4c506db1c879a648ba3f0dc3851c47f597f9ad96 Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114660
>From f52c645a8b0a3664c2520e70b46e248c984037ae Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/chomosuke created
https://github.com/llvm/llvm-project/pull/114660
For some other lsp / linters. They will offer a "Ignore this error for this
line" code actions for warnings. I find that convenient as I don't have to type
the comment myself and copy the name of the diagnost
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114660
>From 3cc27f1a8880e2c31ce19d86e3bb74ce0b82fbec Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Richard Li (chomosuke)
Changes
For some other lsp / linters. They will offer a "Ignore this error for this
line" code actions for warnings. I find that convenient as I don't have to type
the comment myself and copy the name of
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Richard Li (chomosuke)
Changes
For some other lsp / linters. They will offer a "Ignore this error for this
line" code actions for warnings. I find that convenient as I don't have to type
the comment myself and copy the name of the diagno
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/chomosuke updated
https://github.com/llvm/llvm-project/pull/114660
>From 3cc27f1a8880e2c31ce19d86e3bb74ce0b82fbec Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/chomosuke edited
https://github.com/llvm/llvm-project/pull/114660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke closed
https://github.com/llvm/llvm-project/pull/114660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke created
https://github.com/llvm/llvm-project/pull/114661
For some other lsp / linters. They will offer a "Ignore this error for this
line" code actions for warnings. I find that convenient as I don't have to type
the comment myself and copy the name of the diagnost
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/chomosuke updated
https://github.com/llvm/llvm-project/pull/114661
>From 7b70b905e5a9942f592316b407135975977c93cb Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Fri, 16 Aug 2024 13:31:21 +
Subject: [PATCH 01/19] Fixing one error
---
clang-tools-extra/clangd/ClangdServer
https://github.com/chomosuke edited
https://github.com/llvm/llvm-project/pull/114660
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Haojian Wu
Date: 2024-11-02T12:00:18+01:00
New Revision: 67c8b0efbe5c783f39556be2ee841441b50600b5
URL:
https://github.com/llvm/llvm-project/commit/67c8b0efbe5c783f39556be2ee841441b50600b5
DIFF:
https://github.com/llvm/llvm-project/commit/67c8b0efbe5c783f39556be2ee841441b50600b5.diff
LO
@@ -279,7 +281,7 @@ void UseAfterMoveFinder::getDeclRefs(
if (DeclRef && BlockMap->blockContainingStmt(DeclRef) == Block) {
// Ignore uses of a standard smart pointer that don't dereference the
// pointer.
- if (Operator || !isStandardSmartP
https://github.com/5chmidti closed
https://github.com/llvm/llvm-project/pull/113833
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Julian Schmidt
Date: 2024-11-02T12:08:35+01:00
New Revision: 0edaba1b29f8eee011e5fdf387d6c786ec6cb52f
URL:
https://github.com/llvm/llvm-project/commit/0edaba1b29f8eee011e5fdf387d6c786ec6cb52f
DIFF:
https://github.com/llvm/llvm-project/commit/0edaba1b29f8eee011e5fdf387d6c786ec6cb52f.diff
Author: Haojian Wu
Date: 2024-11-02T12:09:52+01:00
New Revision: 2804762e2643c793d12eeabf422b81f4de80ceea
URL:
https://github.com/llvm/llvm-project/commit/2804762e2643c793d12eeabf422b81f4de80ceea
DIFF:
https://github.com/llvm/llvm-project/commit/2804762e2643c793d12eeabf422b81f4de80ceea.diff
LO
https://github.com/SixWeining commented:
Is it the time to remove the FIXME in those tests?
https://github.com/llvm/llvm-project/pull/114511
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114512
>From ef96a6e483030e649d69ffa778c0b0f229bbb739 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:13:29 +0800
Subject: [PATCH 1/2] [LoongArch][Clang] Make the parameter and return value of
{x,}vms
@@ -279,7 +281,7 @@ void UseAfterMoveFinder::getDeclRefs(
if (DeclRef && BlockMap->blockContainingStmt(DeclRef) == Block) {
// Ignore uses of a standard smart pointer that don't dereference the
// pointer.
- if (Operator || !isStandardSmartP
Author: Xi Ruoyao
Date: 2024-11-02T16:19:28+08:00
New Revision: 92daad2eac587cb0592de019cd5f6cbb7c42bb78
URL:
https://github.com/llvm/llvm-project/commit/92daad2eac587cb0592de019cd5f6cbb7c42bb78
DIFF:
https://github.com/llvm/llvm-project/commit/92daad2eac587cb0592de019cd5f6cbb7c42bb78.diff
LOG
https://github.com/chomosuke created
https://github.com/llvm/llvm-project/pull/114663
None
>From 7b70b905e5a9942f592316b407135975977c93cb Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Fri, 16 Aug 2024 13:31:21 +
Subject: [PATCH 01/19] Fixing one error
---
clang-tools-extra/clangd/Clangd
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/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/114639
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke edited
https://github.com/llvm/llvm-project/pull/114663
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kadircet requested changes to this pull request.
thanks I think this LG in terms of module-builder interfaces, but I think we
can make some more changes to implementation to ensure it's easier to maintain
going forward.
speaking of maintenance, @HighCommander4 is definitely
@@ -316,36 +287,169 @@ llvm::Error buildModuleFile(llvm::StringRef ModuleName,
if (Clang->getDiagnostics().hasErrorOccurred())
return llvm::createStringError("Compilation failed");
- BuiltModuleFiles.addModuleFile(ModuleName, Inputs.CompileCommand.Output);
- return llv
https://github.com/kadircet edited
https://github.com/llvm/llvm-project/pull/106683
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -316,36 +287,169 @@ llvm::Error buildModuleFile(llvm::StringRef ModuleName,
if (Clang->getDiagnostics().hasErrorOccurred())
return llvm::createStringError("Compilation failed");
- BuiltModuleFiles.addModuleFile(ModuleName, Inputs.CompileCommand.Output);
- return llv
@@ -316,36 +287,169 @@ llvm::Error buildModuleFile(llvm::StringRef ModuleName,
if (Clang->getDiagnostics().hasErrorOccurred())
return llvm::createStringError("Compilation failed");
- BuiltModuleFiles.addModuleFile(ModuleName, Inputs.CompileCommand.Output);
- return llv
@@ -482,6 +482,42 @@ void func() {
EXPECT_EQ(Result.signatures[0].parameters[0].labelString, "int a");
}
+TEST_F(PrerequisiteModulesTests, ReusablePrerequisiteModulesTest) {
kadircet wrote:
can you also add some tests demonstrating behavior in presence of s
@@ -228,54 +223,30 @@ class StandalonePrerequisiteModules : public
PrerequisiteModules {
return BuiltModuleNames.contains(ModuleName);
}
- void addModuleFile(llvm::StringRef ModuleName,
- llvm::StringRef ModuleFilePath) {
-RequiredModules.emplac
PhilippRados wrote:
@jroelofs can you merge this?
https://github.com/llvm/llvm-project/pull/114410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -228,54 +223,30 @@ class StandalonePrerequisiteModules : public
PrerequisiteModules {
return BuiltModuleNames.contains(ModuleName);
}
- void addModuleFile(llvm::StringRef ModuleName,
- llvm::StringRef ModuleFilePath) {
-RequiredModules.emplac
github-actions[bot] wrote:
@PhilippRados Congratulations on having your first Pull Request (PR) merged
into the LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested by our [build bots](https://lab.llvm.org/buildbot/). If there is a
problem with a b
Author: Philipp Rados
Date: 2024-11-02T08:19:28-07:00
New Revision: ad0a1b90adccd3a9b8fdfbe5923a6ebe9ca2e1a3
URL:
https://github.com/llvm/llvm-project/commit/ad0a1b90adccd3a9b8fdfbe5923a6ebe9ca2e1a3
DIFF:
https://github.com/llvm/llvm-project/commit/ad0a1b90adccd3a9b8fdfbe5923a6ebe9ca2e1a3.diff
https://github.com/jroelofs closed
https://github.com/llvm/llvm-project/pull/114410
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cjappl commented:
for whatever it's worth - LGTM. Leaving to more experienced heads in this area
to actually approve.
Good sleuthing!
https://github.com/llvm/llvm-project/pull/114266
___
cfe-commits mailing list
cfe-commits@lists.
vgvassilev wrote:
@fsfod, can you rebase?
https://github.com/llvm/llvm-project/pull/109362
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -106,6 +106,9 @@ Improvements to clang-tidy
- Improved :program:`run-clang-tidy.py` script. Fixed minor shutdown noise
happening on certain platforms when interrupting the script.
+- Improved :program:`clang-tidy` by adding the option
`CompilationArgsToRemoveRegex`
-
https://github.com/felix642 updated
https://github.com/llvm/llvm-project/pull/111453
From 5cdc27e543b0f4dea3dd3848d974a27614aea713 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?F=C3=A9lix-Antoine=20Constantin?=
Date: Mon, 7 Oct 2024 19:22:44 -0400
Subject: [PATCH 1/2] [clang-tidy] New option to rem
https://github.com/cjappl edited
https://github.com/llvm/llvm-project/pull/114266
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1259,6 +1263,12 @@ class Analyzer {
return true;
}
+bool TraverseStmt(Stmt *Statement) {
+ if (Statement != TrailingRequiresClause)
cjappl wrote:
Could consider leaving a comment here saying why you skip trailing requires
clauses (some
vgvassilev wrote:
@AaronBallman, ping.
https://github.com/llvm/llvm-project/pull/109362
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114663
>From 7b70b905e5a9942f592316b407135975977c93cb Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Fri, 16 Aug 2024 13:31:21 +
Subject: [PATCH 01/20] Fixing one error
---
clang-tools-extra/clangd/ClangdServer
https://github.com/ccotter updated
https://github.com/llvm/llvm-project/pull/114244
>From fd914cc82688b122654d2d7ada72007541b197c0 Mon Sep 17 00:00:00 2001
From: Chris Cotter
Date: Wed, 30 Oct 2024 10:54:49 -0400
Subject: [PATCH 1/5] Add bugprone-sprintf-overlap
---
.../bugprone/BugproneTidyM
ccotter wrote:
While working on #114244, I noticed the use of `auto` in
`memberHasSameNameAsBoundNode`. The only other place I could find using `auto`
with `Nodes.getNode` was in the second file I changed.
https://github.com/llvm/llvm-project/pull/114427
___
https://github.com/chomosuke closed
https://github.com/llvm/llvm-project/pull/114663
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114661
>From 97a830bdf1b3eb1eb352561ef38f95e8baa4e0b9 Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
xry111 wrote:
> Is it the time to remove the FIXME in those tests?
There are several random inconsistencies and I separated the fixes into
multiple PRs as suggested by Xuerui. The FIXME is removed in #114513.
https://github.com/llvm/llvm-project/pull/114511
___
https://github.com/higher-performance edited
https://github.com/llvm/llvm-project/pull/114255
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jroelofs wrote:
sure!
https://github.com/llvm/llvm-project/pull/114410
___
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 `llvm-clang-x86_64-darwin`
running on `doug-worker-3` while building `clang` at step 6
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/23/builds/4425
Here is the r
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114661
>From 581004de1006ce232ad5dbbed9f8af8eb2f312ae Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114514
>From 3d65a59473580f1c0e0b57bdd7fb13c9f40d3077 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:07:17 +0800
Subject: [PATCH 1/5] [LoongArch][clang] Use `signed char` vectors instead of
`char` ve
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114512
>From 3d65a59473580f1c0e0b57bdd7fb13c9f40d3077 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:07:17 +0800
Subject: [PATCH 1/3] [LoongArch][clang] Use `signed char` vectors instead of
`char` ve
Author: Xi Ruoyao
Date: 2024-11-02T15:49:40+08:00
New Revision: b88505414d47ca267f4df8823309264f78935686
URL:
https://github.com/llvm/llvm-project/commit/b88505414d47ca267f4df8823309264f78935686
DIFF:
https://github.com/llvm/llvm-project/commit/b88505414d47ca267f4df8823309264f78935686.diff
LOG
https://github.com/SixWeining approved this pull request.
LGTM. Thanks.
https://github.com/llvm/llvm-project/pull/114510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SixWeining closed
https://github.com/llvm/llvm-project/pull/114510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114511
>From 3d65a59473580f1c0e0b57bdd7fb13c9f40d3077 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:07:17 +0800
Subject: [PATCH 1/2] [LoongArch][clang] Use `signed char` vectors instead of
`char` ve
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114510
>From 3d65a59473580f1c0e0b57bdd7fb13c9f40d3077 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:07:17 +0800
Subject: [PATCH] [LoongArch][clang] Use `signed char` vectors instead of
`char` vector
Author: Xi Ruoyao
Date: 2024-11-02T15:39:15+08:00
New Revision: 96d2196f6f73e5712f1df8cd26de8a12c7f24de4
URL:
https://github.com/llvm/llvm-project/commit/96d2196f6f73e5712f1df8cd26de8a12c7f24de4
DIFF:
https://github.com/llvm/llvm-project/commit/96d2196f6f73e5712f1df8cd26de8a12c7f24de4.diff
LOG
https://github.com/SixWeining closed
https://github.com/llvm/llvm-project/pull/114509
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SixWeining approved this pull request.
https://github.com/llvm/llvm-project/pull/114509
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114511
>From ef96a6e483030e649d69ffa778c0b0f229bbb739 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:13:29 +0800
Subject: [PATCH] [LoongArch][Clang] Make the parameter and return value of
{x,}vmsknz.
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114513
>From 3d65a59473580f1c0e0b57bdd7fb13c9f40d3077 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:07:17 +0800
Subject: [PATCH 1/4] [LoongArch][clang] Use `signed char` vectors instead of
`char` ve
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114514
>From ef96a6e483030e649d69ffa778c0b0f229bbb739 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:13:29 +0800
Subject: [PATCH 1/4] [LoongArch][Clang] Make the parameter and return value of
{x,}vms
https://github.com/xry111 updated
https://github.com/llvm/llvm-project/pull/114513
>From ef96a6e483030e649d69ffa778c0b0f229bbb739 Mon Sep 17 00:00:00 2001
From: Xi Ruoyao
Date: Fri, 1 Nov 2024 00:13:29 +0800
Subject: [PATCH 1/3] [LoongArch][Clang] Make the parameter and return value of
{x,}vms
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin,
=?utf-8?q?F=C3=A9lix-Antoine?= Constantin
Message-ID:
In-Reply-To:
@@ -222,6 +222,12 @@ Changes in existing checks
by adding the option `UseUpperCaseLiteralSuffix` to select the
case of the literal suffix in fixes.
+- Improved :
@@ -132,18 +132,97 @@ is a zero-based file offset, assuming ‘utf-8-unix’
coding."
(lambda (byte &optional _quality _coding-system)
(byte-to-position (1+ byte)
-;;;###autoload
-(defun clang-format-region (start end &optional style assume-file-name)
- "Use clang-
@@ -132,18 +132,97 @@ is a zero-based file offset, assuming ‘utf-8-unix’
coding."
(lambda (byte &optional _quality _coding-system)
(byte-to-position (1+ byte)
-;;;###autoload
-(defun clang-format-region (start end &optional style assume-file-name)
- "Use clang-
@@ -132,18 +132,97 @@ is a zero-based file offset, assuming ‘utf-8-unix’
coding."
(lambda (byte &optional _quality _coding-system)
(byte-to-position (1+ byte)
-;;;###autoload
-(defun clang-format-region (start end &optional style assume-file-name)
- "Use clang-
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ideasman42 requested changes to this pull request.
Note that I'm not a regular clang contributor, I just submitted a small
improvement to clang-format, and maintain some emacs packages in elpa & melpa.
Overall the PR looks like it needs more attention to detail, as far
@@ -132,18 +132,97 @@ is a zero-based file offset, assuming ‘utf-8-unix’
coding."
(lambda (byte &optional _quality _coding-system)
(byte-to-position (1+ byte)
-;;;###autoload
-(defun clang-format-region (start end &optional style assume-file-name)
- "Use clang-
@@ -132,18 +132,97 @@ is a zero-based file offset, assuming ‘utf-8-unix’
coding."
(lambda (byte &optional _quality _coding-system)
(byte-to-position (1+ byte)
-;;;###autoload
-(defun clang-format-region (start end &optional style assume-file-name)
- "Use clang-
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -205,14 +288,60 @@ uses the function `buffer-file-name'."
(delete-file temp-file)
(when (buffer-name temp-buffer) (kill-buffer temp-buffer)
+;;;###autoload
+(defun clang-format-git-diffs (&optional style assume-file-name)
+ "The same as 'clang-format-buffer
@@ -205,14 +288,60 @@ uses the function `buffer-file-name'."
(delete-file temp-file)
(when (buffer-name temp-buffer) (kill-buffer temp-buffer)
+;;;###autoload
+(defun clang-format-git-diffs (&optional style assume-file-name)
ideasman42 wrote:
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ideasman42 edited
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
> Race condition! Will update the other PR.
@dklimkin can you link to "the other PR"? Should we revert this one now?
cc @mydeveloperday @HazardyKnusperkeks
https://github.com/llvm/llvm-project/pull/106398
___
cfe-commits mailing list
cf
owenca wrote:
@ideasman42 can you review this PR? Thanks!
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
goldsteinn wrote:
@luke957 as well also can probably review the elisp code for quality.
https://github.com/llvm/llvm-project/pull/112792
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/113966
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,9 @@
+// RUN: %clang --target=aarch64-pc-windows-msvc -O3 -flto -fuse-ld=lld -### %s
2>&1 | FileCheck %s
+// RUN: %clang --target=aarch64-pc-windows-msvc -O3 -flto=thin -fuse-ld=lld
-### %s 2>&1 | FileCheck %s
+//
MaskRay wrote:
delete `^//$` lines
@@ -67,8 +67,12 @@ XRayArgs::XRayArgs(const ToolChain &TC, const ArgList &Args)
{
false)) {
XRayShared = true;
-// DSO instrumentation is currently limited to x86_64
-if (Triple.getArch() != llvm::Triple::x86_64) {
+// DSO instrumentation is
@@ -67,8 +67,12 @@ XRayArgs::XRayArgs(const ToolChain &TC, const ArgList &Args)
{
false)) {
XRayShared = true;
-// DSO instrumentation is currently limited to x86_64
-if (Triple.getArch() != llvm::Triple::x86_64) {
+// DSO instrumentation is
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/114431
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chomosuke updated
https://github.com/llvm/llvm-project/pull/114661
>From 311c752356617d768f0bdc8599e765ae42272329 Mon Sep 17 00:00:00 2001
From: chomosuke
Date: Sat, 2 Nov 2024 09:18:38 +
Subject: [PATCH] Add quick fix to automatically adds NOLINTNEXTLINE comment
above c
https://github.com/zhaoshiz updated
https://github.com/llvm/llvm-project/pull/113966
>From 5259a3b00fe1b841e8548443036a943b169970dc Mon Sep 17 00:00:00 2001
From: Zhaoshi Zheng
Date: Mon, 28 Oct 2024 14:35:35 -0700
Subject: [PATCH 1/4] [clang][Driver] Allow -fuse-lld=lld-link when lto is
enabl
https://github.com/PhilippRados created
https://github.com/llvm/llvm-project/pull/114687
This is an unfinished implementation of #33528. I have a couple of questions
since I'm new to the codebase:
This warning should be issued for both C and C++. For C++ it is pretty
straightforward since I c
https://github.com/PhilippRados edited
https://github.com/llvm/llvm-project/pull/114687
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
1 - 100 of 135 matches
Mail list logo
