llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Congcong Cai (HerrCai0907)
Changes
`ArraySubscriptExpr` can switch base and idx. For dependent array subscript
access, we should check both base and idx conservatively.
---
Full diff: https://github.com/llvm/llvm-project/pull/128407.
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/128407
`ArraySubscriptExpr` can switch base and idx. For dependent array subscript
access, we should check both base and idx conservatively.
>From ff265c9f01d68b8657d217ba4ea62b77a5775bb5 Mon Sep 17 00:00:00 2001
vbvictor wrote:
Thanks for fast approval!
@HerrCai0907, Could you merge, please.
https://github.com/llvm/llvm-project/pull/128362
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,45 @@
+// RUN: %check_clang_tidy -std=c99 %s bugprone-true-macro %t
+// RUN: %check_clang_tidy -std=c11 %s bugprone-true-macro %t
+// RUN: %check_clang_tidy -std=c17 %s bugprone-true-macro %t
steakhal wrote:
I think you should have a RUN line for a cas
https://github.com/thevinster updated
https://github.com/llvm/llvm-project/pull/128285
>From b8d961acfa2bf17486d63de9481ff46445d6b38f Mon Sep 17 00:00:00 2001
From: Vincent Lee
Date: Fri, 21 Feb 2025 20:33:25 -0800
Subject: [PATCH 1/3] [FatLTO] Detect LLD linker more reliably
---
clang/lib/Dr
HighCommander4 wrote:
If I'm understanding correctly, this patch only completes the keywords
themselves.
SemaCodeComplete also has the ability to produce "snippets", for example
[here](https://searchfox.org/llvm/rev/4e44bd027bf4e490380be770172994561616bd2d/clang/lib/Sema/SemaCodeComplete.cpp#2
https://github.com/RiverDave edited
https://github.com/llvm/llvm-project/pull/128402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RiverDave edited
https://github.com/llvm/llvm-project/pull/128402
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RiverDave edited
https://github.com/llvm/llvm-project/pull/128402
___
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: David Rivera (RiverDave)
Changes
This PR aims to fix `performance-move-const-arg` #126515
## Changes
Enhanced the `performance-move-arg` check in Clang-Tidy to detect cases where
std::move is used
in **ternary expressions whi
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/RiverDave created
https://github.com/llvm/llvm-project/pull/128402
This PR aims to fix `performance-move-const-arg` #126515
## Changes
Enhanced the `performance-move-arg` check in Clang-Tidy to detect cases where
std::move is used
in **ternary expressions which was not bein
HighCommander4 wrote:
> I take a look briefly about `HeuristicResolver`, it looks like not related to
> other Sema parts. Is it possible to move the whole `HeuristicResolver` out of
> `Sema` lib to `AST`?
In future enhancements to `HeuristicResolver`, I would like to implement
optional heuris
https://github.com/HerrCai0907 approved this pull request.
LGTM as temporary solution
But clang-tidy actually do not want to reply on `Sema`. I take a look briefly
about `HeuristicResolver`, it looks like not related to other Sema parts.
Is it possible to move the whole `HeuristicResolver` out
@@ -1101,6 +1107,187 @@ inline void FPOptions::applyChanges(FPOptionsOverride
FPO) {
*this = FPO.applyOverrides(*this);
}
+// The three atomic code-generation options.
+// The canonical (positive) names are:
+// "remote_memory", "fine_grained_memory", and "ignore_denormal
HighCommander4 wrote:
> Note: this is the last remaining use of
> `CXXRecordDecl::lookupDependentName()`. I plan to remove it in a follow-up
> patch.
The follow-up patch is https://github.com/llvm/llvm-project/pull/128392.
https://github.com/llvm/llvm-project/pull/128391
_
HighCommander4 wrote:
Note: this is the last remaining use of `CXXRecordDecl::lookupDependentName()`.
I plan to remove it in a follow-up patch.
https://github.com/llvm/llvm-project/pull/128391
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
ht
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Nathan Ridge (HighCommander4)
Changes
The use replaces CXXRecordDecl::lookupDependentName() which HeuristicResolver
aims to supersede.
---
Full diff: https://github.com/llvm/llvm-project/pull/128391.diff
4 Files Affected:
- (modified)
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/128391
The use replaces CXXRecordDecl::lookupDependentName() which HeuristicResolver
aims to supersede.
>From b44bc0bb4772ff503c8d93430331606d2f2356e3 Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Sat, 22
Icohedron wrote:
> LGTM regarding previous HWASAN crash, which is gone
Thank you for fixing the issue! I will revert the patches then.
https://github.com/llvm/llvm-project/pull/125599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
https://github.com/vitalybuka approved this pull request.
LGTM regarding previous HWASAN crash, which is gone
https://github.com/llvm/llvm-project/pull/125599
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mail
https://github.com/vitalybuka converted_to_draft
https://github.com/llvm/llvm-project/pull/128386
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1,6 +1,5 @@
-; RUN: not llc -verify-machineinstrs -O0 -mtriple=spirv64-unknown-unknown %s
-o /dev/null 2>&1 | FileCheck %s
-; RUN: not llc -verify-machineinstrs -O0 -mtriple=spirv32-unknown-unknown %s
-o /dev/null 2>&1 | FileCheck %s
-; UNSUPPORTED: hwasan
+; RUN: not llc -v
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lld-x86_64-win` running on
`as-worker-93` while building `clang,llvm` at step 7
"test-build-unified-tree-check-all".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/146/builds/2342
Here is the releva
llvmbot wrote:
@llvm/pr-subscribers-hlsl
@llvm/pr-subscribers-clang
Author: Vitaly Buka (vitalybuka)
Changes
This reverts commit 0fe8e70c6609ff86cd40fbb45a85a8ed04c153c2.
---
Patch is 30.02 KiB, truncated to 20.00 KiB below, full version:
https://github.com/llvm/llvm-project/pull/12838
https://github.com/vitalybuka created
https://github.com/llvm/llvm-project/pull/128386
This reverts commit 0fe8e70c6609ff86cd40fbb45a85a8ed04c153c2.
>From 37bc44832ab10781b467c9b4a9e62af594305534 Mon Sep 17 00:00:00 2001
From: Icohedron
Date: Sat, 22 Feb 2025 17:38:18 -0800
Subject: [PATCH] =
https://github.com/jroimartin created
https://github.com/llvm/llvm-project/pull/128385
This is a follow-up to commit f69110dcc973 ("Check for a 'buffer' type instead
of 'buffer-live'.").
In Emacs 29, 'buffer-live' is no longer recognized as a type and generates a
compilation warning. Every f
https://github.com/HerrCai0907 approved this pull request.
https://github.com/llvm/llvm-project/pull/128362
___
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: Roi Martin (jroimartin)
Changes
This is a follow-up to commit f69110dcc973 ("Check for a 'buffer' type instead
of 'buffer-live'.").
In Emacs 29, 'buffer-live' is no longer recognized as a type and generates a
compilation warn
@@ -111,8 +111,13 @@ class SValExplainer : public
FullSValVisitor {
}
std::string VisitSymbolConjured(const SymbolConjured *S) {
-return "symbol of type '" + S->getType().getAsString() +
- "' conjured at statement '" + printStmt(S->getStmt()) + "'";
+std
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
@@ -332,6 +345,14 @@ class UncheckedCallArgsChecker final : public
RawPtrRefCallArgsChecker {
return isUncheckedPtr(QT);
}
+ bool isSafePtr(const CXXRecordDecl *Record) const final {
+return isRefCounted(Record) || isCheckedPtr(Record);
+ }
+
+ bool isSafePtrType
@@ -3668,6 +3668,12 @@ Here are some examples of situations that we warn about
as they *might* be poten
RefCountable* uncounted = counted.get(); // warn
}
+alpha.webkit.UnretainedLocalVarsChecker
+"""
+The goal of this rule is to
@@ -867,18 +868,18 @@ ProgramStateRef createContainerBegin(ProgramStateRef
State,
return setContainerData(State, Cont, CData);
}
-ProgramStateRef createContainerEnd(ProgramStateRef State, const MemRegion
*Cont,
- const Expr *E, QualType T,
@@ -844,7 +845,7 @@ SymbolRef getContainerEnd(ProgramStateRef State, const
MemRegion *Cont) {
return CDataPtr->getEnd();
}
-ProgramStateRef createContainerBegin(ProgramStateRef State,
+ProgramStateRef createContainerBegin(CheckerContext &C, ProgramStateRef State,
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`clang-cmake-x86_64-avx512-win` running on `avx512-intel64-win` while building
`clang` at step 6 "ninja check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/81/builds/5023
Here is the relevant pi
https://github.com/isuckatcs updated
https://github.com/llvm/llvm-project/pull/128265
>From c13cf10fe9f63c4fa361985388ab1ab6c7e55514 Mon Sep 17 00:00:00 2001
From: isuckatcs <65320245+isucka...@users.noreply.github.com>
Date: Mon, 17 Feb 2025 22:50:49 +0100
Subject: [PATCH 1/5] add new check
--
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Congcong Cai (HerrCai0907)
Changes
Tolerate fix-it breaking compilation when functions is used as pointers.
`isReferencedOutsideOfCallExpr` will visit the whole translate unit for each
matched function decls. It will waste lots
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/128383
Tolerate fix-it breaking compilation when functions is used as pointers.
`isReferencedOutsideOfCallExpr` will visit the whole translate unit for each
matched function decls. It will waste lots of cpu time in
https://github.com/isuckatcs updated
https://github.com/llvm/llvm-project/pull/128265
>From c13cf10fe9f63c4fa361985388ab1ab6c7e55514 Mon Sep 17 00:00:00 2001
From: isuckatcs <65320245+isucka...@users.noreply.github.com>
Date: Mon, 17 Feb 2025 22:50:49 +0100
Subject: [PATCH 1/5] add new check
--
@@ -0,0 +1,34 @@
+//===--- TrueMacroCheck.h - clang-tidy --*- C++
-*-===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,84 @@
+// RUN: %check_clang_tidy %s readability-use-numeric-limits %t
+#include
+
+void constants() {
+ // CHECK-MESSAGES: :[[@LINE+2]]:14: warning: The constant -128 is being
utilized. Consider using std::numeric_limits::min() instead
[readability-use-numeric-limit
@@ -1817,7 +1817,7 @@ class TargetLoweringBase {
EVT NewVT) const {
topperc wrote:
Why not move this definition to TargetLowering.cpp? inlining the body of a
virtual function is kind of silly.
https://github.com/llvm/llvm-p
https://github.com/EugeneZelenko approved this pull request.
https://github.com/llvm/llvm-project/pull/128362
___
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: Vitaly Buka (vitalybuka)
Changes
In #123003 make_first_range was applied to temporarily.
---
Full diff: https://github.com/llvm/llvm-project/pull/128372.diff
1 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Checkers/StackAddrEsc
@@ -1817,7 +1817,7 @@ class TargetLoweringBase {
EVT NewVT) const {
RKSimon wrote:
There's an implicit dependency that I'm still trying to track down.
https://github.com/llvm/llvm-project/pull/128167
@@ -332,6 +345,14 @@ class UncheckedCallArgsChecker final : public
RawPtrRefCallArgsChecker {
return isUncheckedPtr(QT);
}
+ bool isSafePtr(const CXXRecordDecl *Record) const final {
+return isRefCounted(Record) || isCheckedPtr(Record);
+ }
+
+ bool isSafePtrType
@@ -862,13 +862,12 @@ void tools::addLTOOptions(const ToolChain &ToolChain,
const ArgList &Args,
const llvm::Triple &Triple = ToolChain.getTriple();
smeenai wrote:
Yeah, I think it's best to keep the old check as well as adding the new one.
https://github.co
https://github.com/t-rasmud approved this pull request.
Left some nitpicks. otherwise LGTM.
https://github.com/llvm/llvm-project/pull/127554
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-com
@@ -3668,6 +3668,12 @@ Here are some examples of situations that we warn about
as they *might* be poten
RefCountable* uncounted = counted.get(); // warn
}
+alpha.webkit.UnretainedLocalVarsChecker
+"""
+The goal of this rule is to
https://github.com/topperc edited
https://github.com/llvm/llvm-project/pull/128167
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/128284
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vitaly Buka
Date: 2025-02-22T13:33:15-08:00
New Revision: d2616cc3926ec0ac73ec547e04b64e045035bd3c
URL:
https://github.com/llvm/llvm-project/commit/d2616cc3926ec0ac73ec547e04b64e045035bd3c
DIFF:
https://github.com/llvm/llvm-project/commit/d2616cc3926ec0ac73ec547e04b64e045035bd3c.diff
L
steakhal wrote:
Ah, I wish we had C++23 already. That would have fixed this too. Thanks!
https://github.com/llvm/llvm-project/pull/128372
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commit
https://github.com/vitalybuka created
https://github.com/llvm/llvm-project/pull/128372
In #123003 make_first_range was applied to temporarily.
>From d3384dec3f6aa0e3d9d6585f8b2553dfcff2d579 Mon Sep 17 00:00:00 2001
From: Vitaly Buka
Date: Sat, 22 Feb 2025 13:32:12 -0800
Subject: [PATCH] =?UTF
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Vitaly Buka (vitalybuka)
Changes
In #123003 make_first_range was applied to temporarily.
---
Full diff: https://github.com/llvm/llvm-project/pull/128372.diff
1 Files Affected:
- (modified) clang/lib/StaticAnalyzer/Che
vitalybuka wrote:
Should be fixed with #128372
https://github.com/llvm/llvm-project/pull/123003
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka closed
https://github.com/llvm/llvm-project/pull/128372
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/128295
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
steakhal wrote:
> Use after scope:
> https://lab.llvm.org/buildbot/#/builders/169/builds/8735
> https://lab.llvm.org/buildbot/#/builders/94/builds/4653/steps/17/logs/stdio
>
Yeey, could you please revert this for me? @vitalybuka
https://github.com/llvm/llvm-project/pull/123003
___
Author: Timm Baeder
Date: 2025-02-22T22:04:44+01:00
New Revision: 6db96c9ecc781c742f546d2863632d44e9c9b435
URL:
https://github.com/llvm/llvm-project/commit/6db96c9ecc781c742f546d2863632d44e9c9b435
DIFF:
https://github.com/llvm/llvm-project/commit/6db96c9ecc781c742f546d2863632d44e9c9b435.diff
L
vitalybuka wrote:
Use after scope:
https://lab.llvm.org/buildbot/#/builders/169/builds/8735
https://lab.llvm.org/buildbot/#/builders/94/builds/4653/steps/17/logs/stdio
https://github.com/llvm/llvm-project/pull/123003
___
cfe-commits mailing list
cfe-c
https://github.com/steakhal created
https://github.com/llvm/llvm-project/pull/128369
Reapply "[analyzer] Delay the checker constructions after parsing" (#128350)
This reverts commit db836edf47f36ed04cab919a7a2c4414f4d0d7e6, as-is.
Depends on #128368
>From 6fd30233a570ace5ccf3f04f649ddd37b
dkolsen-pgi wrote:
I like the idea of changing the assembly format for function types from
`!cir.func` to `!cir.func<(!argType) -> !returnType>`.
That is
1. Easier to parse.
2. Consistent with function types in other MLIR dialects.
3. Consistent with the assembly format for function definition
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 db836edf47f36ed04cab919a7a2c4414f4d0d7e6
ea932d93a47e6747f50768fc232d2c9e9375b6da --e
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 db836edf47f36ed04cab919a7a2c4414f4d0d7e6
6fd30233a570ace5ccf3f04f649ddd37bd4149b2 --e
https://github.com/steakhal edited
https://github.com/llvm/llvm-project/pull/128369
___
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-static-analyzer-1
Author: Balazs Benics (steakhal)
Changes
Reapply "[analyzer] Delay the checker constructions after parsing" (#128350)
This reverts commit db836edf47f36ed04cab919a7a2c4414f4d0d7e6, as-is.
Depends on #128368
---
Patch is 27.
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: Balazs Benics (steakhal)
Changes
Well, yes. It's not pretty.
At least after this we would have a bit more unique pointers than before.
This is for fixing the memory leak diagnosed by:
https://lab.llvm.org/buildbot/#/build
https://github.com/steakhal created
https://github.com/llvm/llvm-project/pull/128368
Well, yes. It's not pretty.
At least after this we would have a bit more unique pointers than before.
This is for fixing the memory leak diagnosed by:
https://lab.llvm.org/buildbot/#/builders/24/builds/5580
An
steakhal wrote:
FYI C23 introduced the `true` and `false` keywords.
https://github.com/llvm/llvm-project/pull/128265
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -14,6 +14,12 @@
#include "hip/hip_version.h"
#endif // __has_include("hip/hip_version.h")
+#ifdef __SPIRV__
+#define __PRIVATE_AS __attribute__((address_space(0)))
arsenm wrote:
Never use numbered address spaces. Unconditionally use
__attribute__((opencl_
https://github.com/HazardyKnusperkeks approved this pull request.
https://github.com/llvm/llvm-project/pull/128287
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca updated
https://github.com/llvm/llvm-project/pull/128287
>From 84cfaec1f830463d2332f14eca9cfbb86919d3d5 Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 21 Feb 2025 21:41:36 -0800
Subject: [PATCH 1/2] [clang-format] Add the C language instead of treating it
like C++
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `lldb-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang,llvm` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/16701
Here is the relevant piece of the bui
thevar1able wrote:
Ping
https://github.com/llvm/llvm-project/pull/103473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay closed
https://github.com/llvm/llvm-project/pull/65451
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: ur4t
Date: 2025-02-22T09:52:53-08:00
New Revision: 62c78919c678915936fcc08212b02db23738dd4d
URL:
https://github.com/llvm/llvm-project/commit/62c78919c678915936fcc08212b02db23738dd4d
DIFF:
https://github.com/llvm/llvm-project/commit/62c78919c678915936fcc08212b02db23738dd4d.diff
LOG: [CM
https://github.com/mikecrowe closed
https://github.com/llvm/llvm-project/pull/126425
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mikecrowe wrote:
Thanks for the reviews. I have been able to turn this check into a plugin
successfully and I will propose that it is included with the nlohmann/json
library.
https://github.com/llvm/llvm-project/pull/126425
___
cfe-commits mailing li
@@ -0,0 +1,72 @@
+//===--- NlohmannJsonExplicitConversionsCheck.cpp - clang-tidy
===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
@@ -0,0 +1,72 @@
+//===--- NlohmannJsonExplicitConversionsCheck.cpp - clang-tidy
===//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM
Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apa
https://github.com/felix642 updated
https://github.com/llvm/llvm-project/pull/126393
From c5d22c2dfba00742892dca6bfc84dd9e70b73d2a Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?F=C3=A9lix-Antoine=20Constantin?=
Date: Sun, 24 Dec 2023 21:01:32 -0500
Subject: [PATCH] [clang-tidy] Fixed clang-tidy rew
https://github.com/antoniofrighetto edited
https://github.com/llvm/llvm-project/pull/127824
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vbvictor wrote:
@EugeneZelenko @HerrCai0907, Small change to docs of some checks.
https://github.com/llvm/llvm-project/pull/128362
___
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-tidy
Author: Baranov Victor (vbvictor)
Changes
Unify doc style for options that use `true` or `false` as default values.
---
Full diff: https://github.com/llvm/llvm-project/pull/128362.diff
3 Files Affected:
- (modified)
clang-tools-extra/docs
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Baranov Victor (vbvictor)
Changes
Unify doc style for options that use `true` or `false` as default values.
---
Full diff: https://github.com/llvm/llvm-project/pull/128362.diff
3 Files Affected:
- (modified)
clang-tools-ext
https://github.com/vbvictor created
https://github.com/llvm/llvm-project/pull/128362
Unify doc style for options that use `true` or `false` as default values.
>From 41807b8f379a19714fb89b2f13bc3af389c44ac0 Mon Sep 17 00:00:00 2001
From: Victor Baranov
Date: Sat, 22 Feb 2025 17:26:52 +0300
Subj
@@ -14,6 +14,12 @@
#include "hip/hip_version.h"
#endif // __has_include("hip/hip_version.h")
+#ifdef __SPIRV__
+#define __PRIVATE_AS __attribute__((address_space(0)))
arsenm wrote:
Using opencl_private is the simpler fix. You are calling code declared and
de
https://github.com/thevar1able updated
https://github.com/llvm/llvm-project/pull/103473
>From 3beb6fc40ccc0b5726b360156063ef35e2b1db3f Mon Sep 17 00:00:00 2001
From: Michael Kolupaev
Date: Tue, 13 Aug 2024 21:16:08 +
Subject: [PATCH 1/2] [libunwind][X86-64] Handle Linux sigreturn trampoline
@@ -14,6 +14,12 @@
#include "hip/hip_version.h"
#endif // __has_include("hip/hip_version.h")
+#ifdef __SPIRV__
+#define __PRIVATE_AS __attribute__((address_space(0)))
AlexVlx wrote:
Acknowledged, but the header was already using numbered address spaces (ROCDL
https://github.com/antoniofrighetto updated
https://github.com/llvm/llvm-project/pull/127824
>From aebd5455e9cf583b9f5a29c68d5217f49c7a49b5 Mon Sep 17 00:00:00 2001
From: Antonio Frighetto
Date: Wed, 19 Feb 2025 16:47:18 +0100
Subject: [PATCH 1/2] [clang][Sema] Propagate qualifiers during deriv
https://github.com/arsenm edited
https://github.com/llvm/llvm-project/pull/128360
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/thevar1able updated
https://github.com/llvm/llvm-project/pull/103473
>From 3beb6fc40ccc0b5726b360156063ef35e2b1db3f Mon Sep 17 00:00:00 2001
From: Michael Kolupaev
Date: Tue, 13 Aug 2024 21:16:08 +
Subject: [PATCH 1/2] [libunwind][X86-64] Handle Linux sigreturn trampoline
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Alex Voicu (AlexVlx)
Changes
Do not hardcode `address_space(5)` (`private`) in the ROCDL interface, as that
breaks SPIRV generation (the latter uses 0). Add test. In the long run we
should stop using ROCDL inline.
---
Patch is 183.37 Ki
llvmbot wrote:
@llvm/pr-subscribers-backend-x86
Author: Alex Voicu (AlexVlx)
Changes
Do not hardcode `address_space(5)` (`private`) in the ROCDL interface, as that
breaks SPIRV generation (the latter uses 0). Add test. In the long run we
should stop using ROCDL inline.
---
Patch is 183
Author: Phoebe Wang
Date: 2025-02-22T20:33:44+08:00
New Revision: 7981c2997c370c6f1b2c9de3d6364e9121136f20
URL:
https://github.com/llvm/llvm-project/commit/7981c2997c370c6f1b2c9de3d6364e9121136f20
DIFF:
https://github.com/llvm/llvm-project/commit/7981c2997c370c6f1b2c9de3d6364e9121136f20.diff
L
@@ -3107,8 +3107,11 @@ Sema::PerformObjectMemberConversion(Expr *From,
/*IgnoreAccess=*/true))
return ExprError();
- return ImpCastExprToType(From, DestType, CK_UncheckedDerivedToBase,
- VK, &BasePath);
+ if (F
https://github.com/antoniofrighetto updated
https://github.com/llvm/llvm-project/pull/127824
>From aebd5455e9cf583b9f5a29c68d5217f49c7a49b5 Mon Sep 17 00:00:00 2001
From: Antonio Frighetto
Date: Wed, 19 Feb 2025 16:47:18 +0100
Subject: [PATCH 1/2] [clang][Sema] Propagate qualifiers during deriv
https://github.com/antoniofrighetto updated
https://github.com/llvm/llvm-project/pull/127824
>From aebd5455e9cf583b9f5a29c68d5217f49c7a49b5 Mon Sep 17 00:00:00 2001
From: Antonio Frighetto
Date: Wed, 19 Feb 2025 16:47:18 +0100
Subject: [PATCH 1/2] [clang][Sema] Propagate qualifiers during deriv
1 - 100 of 163 matches
Mail list logo