https://github.com/dtcxzyw approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/102022
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yingopq wrote:
@MaskRay please help review, I have added test and update code `MipsMsa`.
Thanks!
https://github.com/llvm/llvm-project/pull/99615
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cf
https://github.com/hekota updated
https://github.com/llvm/llvm-project/pull/102275
>From 340834e9c1ea12c8de4ee9f5a6b7c0191e96f489 Mon Sep 17 00:00:00 2001
From: Helena Kotas
Date: Tue, 6 Aug 2024 23:19:20 -0700
Subject: [PATCH 1/2] [HLSL] Add list of exported functions as named metadata
node '
https://github.com/hekota edited
https://github.com/llvm/llvm-project/pull/102275
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hekota updated
https://github.com/llvm/llvm-project/pull/102275
>From 340834e9c1ea12c8de4ee9f5a6b7c0191e96f489 Mon Sep 17 00:00:00 2001
From: Helena Kotas
Date: Tue, 6 Aug 2024 23:19:20 -0700
Subject: [PATCH 1/3] [HLSL] Add list of exported functions as named metadata
node '
https://github.com/hekota ready_for_review
https://github.com/llvm/llvm-project/pull/102275
___
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-codegen
Author: Helena Kotas (hekota)
Changes
Adds list of exported functions as named metadata node 'dx.exports`.
During CodeGen each exported functions is marked with an "hlsl.export"
attribute. Based on these attributes the DXILTranslateMetada
llvmbot wrote:
@llvm/pr-subscribers-backend-directx
Author: Helena Kotas (hekota)
Changes
Adds list of exported functions as named metadata node 'dx.exports`.
During CodeGen each exported functions is marked with an "hlsl.export"
attribute. Based on these attributes the DXILTranslateMeta
camel-cdr wrote:
Are the vrgather.vv numbers correct? Usually LMUL>1 vrgather is implemented by
aplying a LMUL=1 vrgather LMUL^2 times. Since the LMUL=1 vrgather.vv takes a
single cycle, I would've expected a 4 cycle LMUL=2 vrgather.vv instead of 12.
12 cycles is less than one element per cycle
glandium wrote:
> We (Chrome) no longer need a revert - we patched the NDK locally.
Patching the NDK is not a solution that is nice everywhere, though. And the new
warning is an error that can't be disabled.
https://github.com/llvm/llvm-project/pull/70307
__
https://github.com/a-tarasyuk deleted
https://github.com/llvm/llvm-project/pull/101853
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -6719,6 +6719,16 @@ QualType Sema::CheckVectorConditionalTypes(ExprResult
&Cond, ExprResult &LHS,
: UsualArithmeticConversions(LHS, RHS, QuestionLoc,
ACK_Conditional);
+if (ResultElementTy.isNull()) {
+ Diag(Que
@@ -8052,10 +8061,7 @@ NamedDecl *Sema::ActOnVariableDeclarator(
}
// Special handling of variable named 'main'.
- if (Name.getAsIdentifierInfo() && Name.getAsIdentifierInfo()->isStr("main")
&&
- NewVD->getDeclContext()->getRedeclContext()->isTranslationUnit() &&
-
https://github.com/nikic approved this pull request.
https://github.com/llvm/llvm-project/pull/102274
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/danix800 edited
https://github.com/llvm/llvm-project/pull/102152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/danix800 edited
https://github.com/llvm/llvm-project/pull/102152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
owenca wrote:
> When you say rename, do you mean deprecate `BinPackParameters` like in these
> changes
> [6739bb5](https://github.com/llvm/llvm-project/commit/6739bb5006bc28e2bdbdb2326eb2c957546634aa)
> but using `BreakParameters` and `Never`/`OnePerLine`/`Always` for the names?
Yes, deprecat
https://github.com/asi-sc created
https://github.com/llvm/llvm-project/pull/102285
Syntacore SCR5 is an entry-level Linux-capable 32/64-bit RISC-V processor core.
Overview: https://syntacore.com/products/scr5
Scheduling model will be added in a subsequent PR.
Co-authored-by: Dmitrii Petrov
Co
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
@llvm/pr-subscribers-clang-driver
Author: Anton Sidorenko (asi-sc)
Changes
Syntacore SCR5 is an entry-level Linux-capable 32/64-bit RISC-V processor core.
Overview: https://syntacore.com/products/scr5
Scheduling model will be added in a s
mydeveloperday wrote:
Whilst I can understand BinPackParameters being deprecated, as its one of the
original options I do worry a little about just how much impact such a
deprecation might have... it needs to be completely seemless because there is
ALOT of documentation references to its use o
https://github.com/ChuanqiXu9 milestoned
https://github.com/llvm/llvm-project/pull/102287
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 created
https://github.com/llvm/llvm-project/pull/102287
Reland https://github.com/llvm/llvm-project/pull/75912
The differences of this PR between
https://github.com/llvm/llvm-project/pull/75912 are:
- Fixed a regression in `Decl::isInAnotherModuleUnit()` in Decl
https://github.com/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/102287
___
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-modules
Author: Chuanqi Xu (ChuanqiXu9)
Changes
Reland https://github.com/llvm/llvm-project/pull/75912
The differences of this PR between
https://github.com/llvm/llvm-project/pull/75912 are:
- Fixed a regression in `Decl::isInAnotherModuleUnit()
https://github.com/mydeveloperday approved this pull request.
https://github.com/llvm/llvm-project/pull/102261
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mydeveloperday requested changes to this pull request.
I'm of the opinion if you are changing tests then something is wrong.
https://github.com/llvm/llvm-project/pull/102078
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
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 89db3bbd27ddc5ec980799c987dafd167c5a4564
7daf703b8d2602bd13d9eb9a9e6c9487205427b0 --e
https://github.com/nikic approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/101114
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Gitspike updated
https://github.com/llvm/llvm-project/pull/102152
>From c74cea916b5efe739de8a4dc6f63d583f5a46da3 Mon Sep 17 00:00:00 2001
From: hehouhua
Date: Wed, 7 Aug 2024 11:55:30 +0800
Subject: [PATCH 1/2] [clang][ASTMatcher] Add matches for StringLiteral which
matches
https://github.com/Gitspike updated
https://github.com/llvm/llvm-project/pull/102152
>From 53182588d50cc3f29621eeb7dba0e5c30ed4bfa3 Mon Sep 17 00:00:00 2001
From: hehouhua
Date: Wed, 7 Aug 2024 11:55:30 +0800
Subject: [PATCH] [clang][ASTMatcher] Add matches for StringLiteral which
matches lite
https://github.com/Gitspike updated
https://github.com/llvm/llvm-project/pull/102152
>From 53182588d50cc3f29621eeb7dba0e5c30ed4bfa3 Mon Sep 17 00:00:00 2001
From: hehouhua
Date: Wed, 7 Aug 2024 11:55:30 +0800
Subject: [PATCH] [clang][ASTMatcher] Add matches for StringLiteral which
matches lite
@@ -2376,6 +2376,12 @@ NamedDecl *Sema::LazilyCreateBuiltin(IdentifierInfo *II,
unsigned ID,
FunctionDecl *New = CreateBuiltin(II, R, ID, Loc);
RegisterLocallyScopedExternCDecl(New, S);
+ // Builtin functions shouldn't be owned by any module.
+ if (New->hasOwningModule(
https://github.com/SpencerAbson edited
https://github.com/llvm/llvm-project/pull/101480
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Oliver Stannard
Date: 2024-08-07T10:21:10+01:00
New Revision: 96d824d935d631a4c28133f9f8f1f583eefd040d
URL:
https://github.com/llvm/llvm-project/commit/96d824d935d631a4c28133f9f8f1f583eefd040d
DIFF:
https://github.com/llvm/llvm-project/commit/96d824d935d631a4c28133f9f8f1f583eefd040d.dif
https://github.com/ostannard closed
https://github.com/llvm/llvm-project/pull/102126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
gonzalobg wrote:
@Artem-B this LGTM
https://github.com/llvm/llvm-project/pull/99646
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/gonzalobg approved this pull request.
https://github.com/llvm/llvm-project/pull/99646
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -2172,7 +2172,7 @@ let SVETargetGuard = InvalidMode, SMETargetGuard = "sme2"
in {
def SVFCLAMP_X4 : SInst<"svclamp[_single_{d}_x4]", "44dd", "hfd",
MergeNone, "aarch64_sve_fclamp_single_x4", [IsStreaming], []>;
}
-let SVETargetGuard = InvalidMode, SMETargetGuar
@@ -0,0 +1,49 @@
+// RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sve
-target-feature +sve2 -verify -verify-ignore-unexpected=error,note -emit-llvm
-o - %s
SpencerAbson wrote:
Will do :+1:
https://github.com/llvm/llvm-project/pull/101480
__
https://github.com/CarolineConcatto updated
https://github.com/llvm/llvm-project/pull/97755
>From 22f5bb7cab1673632f1fa5438a35f861c16d63b2 Mon Sep 17 00:00:00 2001
From: Caroline Concatto
Date: Thu, 4 Jul 2024 17:10:36 +
Subject: [PATCH 1/3] [Clang][LLVM][AArch64] Add intrinsic for LUTI4 SM
https://github.com/MichelleCDjunaidi created
https://github.com/llvm/llvm-project/pull/102299
This checks that classes/structs inheriting from
``std::enable_shared_from_this`` derive it with the ``public`` access
specifier, so it prevents crashes due to ``std::make_shared`` and
``shared_from_
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
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: None (MichelleCDjunaidi)
Changes
This checks that classes/structs inheriting from
``std::enable_shared_from_this`` derive it with the ``public`` access
specifier, so it prevents crashes due to ``std::make_shared`` and
``share
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: None (MichelleCDjunaidi)
Changes
This checks that classes/structs inheriting from
``std::enable_shared_from_this`` derive it with the ``public`` access
specifier, so it prevents crashes due to ``std::make_shared`` and
``shared_from_
tbaederr wrote:
@vikramRH Do you need someone else to merge this for you?
https://github.com/llvm/llvm-project/pull/101126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/rymiel approved this pull request.
https://github.com/llvm/llvm-project/pull/102261
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vikramRH wrote:
### Merge activity
* **Aug 7, 6:38 AM EDT**: @vikramRH started a stack merge that includes this
pull request via
[Graphite](https://app.graphite.dev/github/pr/llvm/llvm-project/101126).
https://github.com/llvm/llvm-project/pull/101126
_
Author: Vikram Hegde
Date: 2024-08-07T16:11:08+05:30
New Revision: 77534291fcbd2c784c54e39a60895e4f60f19742
URL:
https://github.com/llvm/llvm-project/commit/77534291fcbd2c784c54e39a60895e4f60f19742
DIFF:
https://github.com/llvm/llvm-project/commit/77534291fcbd2c784c54e39a60895e4f60f19742.diff
https://github.com/vikramRH closed
https://github.com/llvm/llvm-project/pull/101126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vikramRH wrote:
> @vikramRH Do you need someone else to merge this for you?
sorry for the delay, merged.
https://github.com/llvm/llvm-project/pull/101126
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/li
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/102302
We were not doing the final Neg here.
>From 0b1a9b49382802a06fad9ac149e2f2a20539429e Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 7 Aug 2024 12:25:06 +0200
Subject: [PATCH] [clang][I
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Timm Baeder (tbaederr)
Changes
We were not doing the final Neg here.
---
Full diff: https://github.com/llvm/llvm-project/pull/102302.diff
2 Files Affected:
- (modified) clang/lib/AST/Interp/Compiler.cpp (+11-5)
- (modified) clang/test/
@@ -680,7 +680,7 @@
DEFINE_LIBUNWIND_FUNCTION(__libunwind_Registers_arm64_jumpto)
ldrx16, [x0, #0x0F8]
ldpx0, x1, [x0, #0x000] // restore x0,x1
movsp,x16 // restore sp
- retx30// jump to pc
john-
https://github.com/vikramRH closed
https://github.com/llvm/llvm-project/pull/72607
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vikramRH wrote:
closing this, since its handled via
https://github.com/llvm/llvm-project/pull/101126
https://github.com/llvm/llvm-project/pull/72607
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
prj- wrote:
I'm sorry I'm not familiar with LLVM release cycle, but is it too late for this
to make it into the `release/19.x` branch?
https://github.com/llvm/llvm-project/pull/102261
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lis
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/102302
>From 6de1136c9806d3b545b586d91ee56a934cff1669 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 7 Aug 2024 12:25:06 +0200
Subject: [PATCH] [clang][Interp] Fix BooleanToSignedIntegral cast
https://github.com/Fznamznon created
https://github.com/llvm/llvm-project/pull/102304
Fixes https://github.com/llvm/llvm-project/issues/102248
>From ebeb9264e8a4e6d57a4573376248191f64ad99e7 Mon Sep 17 00:00:00 2001
From: "Podchishchaeva, Mariya"
Date: Wed, 7 Aug 2024 04:04:09 -0700
Subject: [P
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Mariya Podchishchaeva (Fznamznon)
Changes
Fixes https://github.com/llvm/llvm-project/issues/102248
---
Full diff: https://github.com/llvm/llvm-project/pull/102304.diff
3 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+1)
- (
vgvassilev wrote:
ping.
https://github.com/llvm/llvm-project/pull/98138
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin edited
https://github.com/llvm/llvm-project/pull/102304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/cor3ntin commented:
LGTM.
I think it is reasonable to backport that (in which case we don't need the
changelog entry)
https://github.com/llvm/llvm-project/pull/102304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
@@ -1,9 +1,15 @@
// RUN: %clang_cc1 %s -triple x86_64 --embed-dir=%S/Inputs -emit-llvm -o - |
FileCheck %s
+// CHECK: @.compoundliteral = internal global [2 x i8] c"jk"
// CHECK: @__const._Z3fooi.ca = private unnamed_addr constant [3 x i32] [i32
0, i32 106, i32 107], align 4
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/102299
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -139,6 +140,8 @@ class BugproneModule : public ClangTidyModule {
"bugprone-inaccurate-erase");
CheckFactories.registerCheck(
"bugprone-incorrect-enable-if");
+CheckFactories.registerCheck(
+"bugprone-public-enable-shared-from-this");
-
https://github.com/PiotrZSL requested changes to this pull request.
I'm fine with idea behind this check. Just few fixes are needed.
https://github.com/llvm/llvm-project/pull/102299
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,45 @@
+// RUN: %check_clang_tidy %s bugprone-public-enable-shared-from-this %t -- --
+
+namespace std {
+
+template class enable_shared_from_this {};
+
PiotrZSL wrote:
std namespace should end here
https://github.com/llvm/llvm-project/pull/102299
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,33 @@
+//===--- PublicEnableSharedFromThisCheck.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,45 @@
+// RUN: %check_clang_tidy %s bugprone-public-enable-shared-from-this %t -- --
+
+namespace std {
+
+template class enable_shared_from_this {};
+
+class BadExample : enable_shared_from_this {};
+// CHECK-MESSAGES: :[[@LINE-1]]:7: warning: class BadExa
@@ -0,0 +1,45 @@
+// RUN: %check_clang_tidy %s bugprone-public-enable-shared-from-this %t -- --
+
+namespace std {
+
+template class enable_shared_from_this {};
+
+class BadExample : enable_shared_from_this {};
+// CHECK-MESSAGES: :[[@LINE-1]]:7: warning: class BadExa
@@ -0,0 +1,26 @@
+.. title:: clang-tidy - bugprone-public-enable-shared-from-this
+
+bugprone-public-enable-shared-from-this
+===
+
+Checks that classes/structs inheriting from ``std::enable_shared_from_this``
derive it with the ``public`` acce
@@ -98,6 +98,11 @@ Improvements to clang-tidy
New checks
^^
+- New :doc:`bugprone-public-enable-shared-from-this
+ ` check.
+
+ Checks if a class deriving from enable_shared_from_this has public access
specifiers, because otherwise the program will crash when shared
github-actions[bot] wrote:
⚠️ We detected that you are using a GitHub private e-mail address to contribute
to the repo. Please turn off [Keep my email addresses
private](https://github.com/settings/emails) setting in your account. See
[LLVM
Discourse](https://discourse.llvm.org/t/hidden-email
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 4745bb3e7db6d9cf5588171a433f534e5b488c70
7554c5d6fb2bc442fe3d2e00826ea5a0e34e432b --e
@@ -139,6 +140,8 @@ class BugproneModule : public ClangTidyModule {
"bugprone-inaccurate-erase");
CheckFactories.registerCheck(
"bugprone-incorrect-enable-if");
+CheckFactories.registerCheck(
+"bugprone-public-enable-shared-from-this");
-
https://github.com/AaronBallman edited
https://github.com/llvm/llvm-project/pull/102304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/AaronBallman approved this pull request.
Changes LGTM as far as they go, but some additional test coverage is requested.
https://github.com/llvm/llvm-project/pull/102304
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
@@ -1,9 +1,15 @@
// RUN: %clang_cc1 %s -triple x86_64 --embed-dir=%S/Inputs -emit-llvm -o - |
FileCheck %s
+// CHECK: @.compoundliteral = internal global [2 x i8] c"jk"
// CHECK: @__const._Z3fooi.ca = private unnamed_addr constant [3 x i32] [i32
0, i32 106, i32 107], align 4
@@ -0,0 +1,45 @@
+// RUN: %check_clang_tidy %s bugprone-public-enable-shared-from-this %t -- --
+
+namespace std {
+
+template class enable_shared_from_this {};
+
+class BadExample : enable_shared_from_this {};
+// CHECK-MESSAGES: :[[@LINE-1]]:7: warning: class BadExa
@@ -0,0 +1,26 @@
+.. title:: clang-tidy - bugprone-public-enable-shared-from-this
+
+bugprone-public-enable-shared-from-this
+===
+
+Checks that classes/structs inheriting from ``std::enable_shared_from_this``
derive it with the ``public`` acce
@@ -98,6 +98,11 @@ Improvements to clang-tidy
New checks
^^
+- New :doc:`bugprone-public-enable-shared-from-this
+ ` check.
+
+ Checks if a class deriving from enable_shared_from_this has public access
specifiers, because otherwise the program will crash when shared
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
Author: Timm Baeder
Date: 2024-08-07T13:52:00+02:00
New Revision: f05e8186a439cf538f383dfbde68eca364a5acec
URL:
https://github.com/llvm/llvm-project/commit/f05e8186a439cf538f383dfbde68eca364a5acec
DIFF:
https://github.com/llvm/llvm-project/commit/f05e8186a439cf538f383dfbde68eca364a5acec.diff
L
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/102302
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
@@ -0,0 +1,45 @@
+//===--- PublicEnableSharedFromThisCheck.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-Identif
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/102299
___
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 `lldb-arm-ubuntu` running
on `linaro-lldb-arm-ubuntu` while building `clang` at step 6 "test".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/18/builds/1875
Here is the relevant piece of the build log
Author: Michael Maitland
Date: 2024-08-07T08:30:42-04:00
New Revision: 0c25f85e5b88102363c0cd55e1946053d5827e99
URL:
https://github.com/llvm/llvm-project/commit/0c25f85e5b88102363c0cd55e1946053d5827e99
DIFF:
https://github.com/llvm/llvm-project/commit/0c25f85e5b88102363c0cd55e1946053d5827e99.di
https://github.com/michaelmaitland closed
https://github.com/llvm/llvm-project/pull/102022
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/DanielKristofKiss updated
https://github.com/llvm/llvm-project/pull/101978
>From 4afadb9122c982c63f2b067661548a2c063590a5 Mon Sep 17 00:00:00 2001
From: Daniel Kiss
Date: Mon, 5 Aug 2024 13:01:06 +0200
Subject: [PATCH] [Arm][AArch64][Clang] Respect function's branch protectio
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-ppc64le-linux`
running on `ppc64le-sanitizer` while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/72/builds/1984
Here is the relevant piece of the bui
https://github.com/CarolineConcatto edited
https://github.com/llvm/llvm-project/pull/101480
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/CarolineConcatto approved this pull request.
Thank you Spencer for the work.
I left some comments about the sve2.1 feature flags being used for the sve2.
But I dont think you need to change in this patch. Because it is unrelated
Maybe you create another patch a NFC to move all
@@ -1,10 +1,10 @@
// NOTE: Assertions have been autogenerated by utils/update_cc_test_checks.py
// REQUIRES: aarch64-registered-target
-// RUN: %clang_cc1 -triple aarch64 -target-feature +sve -target-feature +sve2
-target-feature +sve2p1 -target-feature +b16b16 -disable-O0-optn
1 - 100 of 520 matches
Mail list logo
