https://github.com/tianleliu edited
https://github.com/llvm/llvm-project/pull/127442
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
mylai-mtk wrote:
> Can you update the description to mention which PR makes RISCV targets
> inspect -fcf-protection=?
Added PR links. Thanks!
https://github.com/llvm/llvm-project/pull/127616
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
https://github.com/tianleliu updated
https://github.com/llvm/llvm-project/pull/127442
>From ad4d2857220e5e5b31cd66a79eac7d8dca438864 Mon Sep 17 00:00:00 2001
From: tianleli
Date: Mon, 17 Feb 2025 11:24:10 +0800
Subject: [PATCH 1/3] [SPGO][Driver] Add lto-sample-profile option for lld-link
SPGO
https://github.com/chandraghale created
https://github.com/llvm/llvm-project/pull/127740
None
>From cf392f05f9499fd0621ffec91a3b852d4b91820b Mon Sep 17 00:00:00 2001
From: Chandra Ghale
Date: Tue, 18 Feb 2025 21:24:22 -0600
Subject: [PATCH] Inital support for privavate variable reduction
---
https://github.com/Ami-zhang updated
https://github.com/llvm/llvm-project/pull/127555
>From c6df3aa9b9769d096d2cd63fc2353665ec9edbd1 Mon Sep 17 00:00:00 2001
From: Ami-zhang
Date: Tue, 18 Feb 2025 10:00:25 +0800
Subject: [PATCH] [clang][LoongArch] Add OHOS target
Add support for OHOS on loonga
https://github.com/Artem-B approved this pull request.
LGTM.
Do you need help merging the patch?
https://github.com/llvm/llvm-project/pull/127187
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/
https://github.com/mxms0 edited https://github.com/llvm/llvm-project/pull/127698
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,30 @@
+.. title:: clang-tidy - bugprone-unintended-char-ostream-output
+
+bugprone-unintended-char-ostream-output
+===
+
+Finds unintended character output from `unsigned char` and `signed char` to an
EugeneZelenko wr
@@ -721,6 +721,8 @@ DecodeStatus RISCVDisassembler::getInstruction32(MCInst
&MI, uint64_t &Size,
"Qualcomm uC Conditional Move custom opcode table");
TRY_TO_DECODE_FEATURE(RISCV::FeatureVendorXqciint, DecoderTableXqciint32,
"Qu
https://github.com/keith updated
https://github.com/llvm/llvm-project/pull/127734
>From b8c294fcd30153842e1c0741fc623f45317618b1 Mon Sep 17 00:00:00 2001
From: Keith Smiley
Date: Wed, 19 Feb 2025 01:54:42 +
Subject: [PATCH] [clang][Tooling] Support relative directory in compilation
databas
https://github.com/t-rasmud edited
https://github.com/llvm/llvm-project/pull/127570
___
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/127442
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -87,6 +92,31 @@ class RawPtrRefMemberChecker
}
}
+ void visitObjCDecl(const ObjCContainerDecl *CD) const {
+if (auto *ID = dyn_cast(CD)) {
t-rasmud wrote:
Ah ok. I was wondering if we should check for dullness before dereferencing
`ID` in the n
ergawy wrote:
@mjklemm suggested to add a warning that the pass is still experimental, which
I think is a good idea. However, I am wondering what the best place for that
warning would be? I prefer not to do that in
`CodeGenAction::beginSourceFileAction()` (where we inspect the flag and
actual
Author: tianleliu
Date: 2025-02-19T12:45:45+08:00
New Revision: 6c39ee717f03a0fe28f563d525fa5aff09804ba8
URL:
https://github.com/llvm/llvm-project/commit/6c39ee717f03a0fe28f563d525fa5aff09804ba8
DIFF:
https://github.com/llvm/llvm-project/commit/6c39ee717f03a0fe28f563d525fa5aff09804ba8.diff
LOG
https://github.com/tianleliu closed
https://github.com/llvm/llvm-project/pull/127442
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
bd1976bris wrote:
> Some high level comments:
> * Should we just call it distributedLTO, or DLTO? Feel like we can drop the
> thin part for less typing, and from the user's point of view, using thinLTO
> infrastructure is just implementation details.
Thanks! I'll discuss this and get back to y
@@ -87,6 +92,31 @@ class RawPtrRefMemberChecker
}
}
+ void visitObjCDecl(const ObjCContainerDecl *CD) const {
+if (auto *ID = dyn_cast(CD)) {
rniwa wrote:
oh, but this if statement does that already, right? as in, this if will only
evaluate to tru
https://github.com/ShashwathiNavada updated
https://github.com/llvm/llvm-project/pull/125643
>From 0aebcd7119fbcd51154c5d9706752e8ff3f041bc Mon Sep 17 00:00:00 2001
From: ShashwathiNavada
Date: Tue, 4 Feb 2025 00:16:09 -0600
Subject: [PATCH 1/6] Adding diagnostics for unsupported option
---
c
@@ -87,6 +92,31 @@ class RawPtrRefMemberChecker
}
}
+ void visitObjCDecl(const ObjCContainerDecl *CD) const {
+if (auto *ID = dyn_cast(CD)) {
+ for (auto *Ivar : ID->ivars())
+visitIvarDecl(CD, Ivar);
+ return;
+}
+if (auto *ID = dyn_cast(C
@@ -346,6 +346,24 @@ BUILTIN(__builtin_amdgcn_endpgm, "v", "nr")
BUILTIN(__builtin_amdgcn_get_fpenv, "WUi", "n")
BUILTIN(__builtin_amdgcn_set_fpenv, "vWUi", "n")
+//===--===//
+
+// Wave Reduction builtins.
+
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/127720
>From b69bb465a24f2175f2f9f91f220252d3bcb27bde Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 19 Feb 2025 07:38:37 +0800
Subject: [PATCH 1/2] [clang-tidy]add new check
bugprone-unintended-char-ostre
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
anutosh491 wrote:
Maybe @vgvassilev might know more about this behaviour and if it is expected.
For some context you can read the discussion from here
https://github.com/llvm/llvm-project/pull/127087#issuecomment-2664564232
https://github.com/llvm/llvm-project/pull/127087
_
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-aarch64-linux`
running on `sanitizer-buildbot8` while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/51/builds/3
Here is the relevant piece of the
Author: Kazu Hirata
Date: 2025-02-18T23:18:08-08:00
New Revision: 8f41d28d89ee287d0f5a6518116ab316be2657b8
URL:
https://github.com/llvm/llvm-project/commit/8f41d28d89ee287d0f5a6518116ab316be2657b8
DIFF:
https://github.com/llvm/llvm-project/commit/8f41d28d89ee287d0f5a6518116ab316be2657b8.diff
L
@@ -0,0 +1,30 @@
+.. title:: clang-tidy - bugprone-unintended-char-ostream-output
+
+bugprone-unintended-char-ostream-output
+===
+
+Finds unintended character output from ``unsigned char`` and ``signed char``
to an
+``ostream``.
+
+Normally, w
@@ -0,0 +1,69 @@
+// RUN: %check_clang_tidy %s bugprone-unintended-char-ostream-output %t
+
+namespace std {
+
+template class basic_ostream {
+public:
+ basic_ostream &operator<<(int);
+ basic_ostream &operator<<(unsigned int);
+};
+
+template
+basic_ostream &operator<<(basic
vitalybuka wrote:
> @metaflow @vitalybuka I've checked in
> [8f41d28](https://github.com/llvm/llvm-project/commit/8f41d28d89ee287d0f5a6518116ab316be2657b8)
> to fix warnings. Thanks!
Thank You!
https://github.com/llvm/llvm-project/pull/127063
___
cf
https://github.com/vitalybuka updated
https://github.com/llvm/llvm-project/pull/127078
>From a831b96cc2eb4b65e7157ee3cb519cf9b4a6af76 Mon Sep 17 00:00:00 2001
From: Vitaly Buka
Date: Thu, 13 Feb 2025 07:42:46 -0800
Subject: [PATCH] Revert "Revert "[libclang] Always Dup in
createRef(StringRef)"
https://github.com/HighCommander4 created
https://github.com/llvm/llvm-project/pull/127757
…
>From d7f83b154b17856b1de2d97cf0a3aca72dd0379c Mon Sep 17 00:00:00 2001
From: Nathan Ridge
Date: Wed, 19 Feb 2025 02:25:16 -0500
Subject: [PATCH] [clangd] Avoid round-trip from SourceLocation to
clang
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Nathan Ridge (HighCommander4)
Changes
…
---
Full diff: https://github.com/llvm/llvm-project/pull/127757.diff
3 Files Affected:
- (modified) clang-tools-extra/clangd/CollectMacros.cpp (+6-3)
- (modified) clang-tools-extra/cl
HighCommander4 wrote:
> (For example `ClangdModules` was an action we were trying to take in this
> direction. It actually aims to provide people infrastructure to implement
> their desired functionality in clangd, while limiting resource and
> maintenance costs to only that specific feature a
llvmbot wrote:
@llvm/pr-subscribers-clangd
Author: Nathan Ridge (HighCommander4)
Changes
…
---
Full diff: https://github.com/llvm/llvm-project/pull/127757.diff
3 Files Affected:
- (modified) clang-tools-extra/clangd/CollectMacros.cpp (+6-3)
- (modified) clang-tools-extra/clangd/Collec
https://github.com/tbaederr created
https://github.com/llvm/llvm-project/pull/127759
This _can_ happen with non-pointers, but we shouldn't diagnose it in that case.
>From 1b4f4b78741e297e8d257af427206f3dbc3501d2 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Wed, 19 Feb 202
kazutakahirata wrote:
@metaflow @vitalybuka I've checked in 8f41d28d89ee287d0f5a6518116ab316be2657b8
to fix warnings. Thanks!
https://github.com/llvm/llvm-project/pull/127063
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `sanitizer-x86_64-linux`
running on `sanitizer-buildbot1` while building `clang` at step 2 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/66/builds/10094
Here is the relevant piece of the
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`ppc64le-lld-multistage-test` running on `ppc64le-lld-multistage-test` while
building `clang` at step 12 "build-stage2-unified-tree".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/168/builds/8866
He
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/67095
>From 1bb62210a2ec5342111a0407cbfa8a73d761357f Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Fri, 22 Sep 2023 08:42:05 +0200
Subject: [PATCH] [clang][TSA] Make RequiresCapability a DeclOrTyp
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
https://github.com/Flandini updated
https://github.com/llvm/llvm-project/pull/127702
>From 12791f2c89f7e42bd261ac573c2497857c42b6f3 Mon Sep 17 00:00:00 2001
From: Michael Flanders
Date: Tue, 18 Feb 2025 15:56:13 -0600
Subject: [PATCH 1/4] [analyzer] Do list initialization for CXXNewExpr with
i
mjklemm wrote:
One warning per source file is what is being done for -fopenmp at the moment.
So, it should be fine to do that.
https://github.com/llvm/llvm-project/pull/126026
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
anutosh491 wrote:
Hmmm confused !
So does it really come down to the flag responsible for enabling assertions.
`LLVM_ENABLE_ASSERTIONS=ON`
https://github.com/llvm/llvm-project/pull/127087
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:/
Flandini wrote:
Going to add tests for placement new and user-defined placement new operator
with list initializers also
https://github.com/llvm/llvm-project/pull/127702
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cg
https://github.com/Flandini updated
https://github.com/llvm/llvm-project/pull/127702
>From 12791f2c89f7e42bd261ac573c2497857c42b6f3 Mon Sep 17 00:00:00 2001
From: Michael Flanders
Date: Tue, 18 Feb 2025 15:56:13 -0600
Subject: [PATCH 1/5] [analyzer] Do list initialization for CXXNewExpr with
i
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
@@ -346,6 +346,24 @@ BUILTIN(__builtin_amdgcn_endpgm, "v", "nr")
BUILTIN(__builtin_amdgcn_get_fpenv, "WUi", "n")
BUILTIN(__builtin_amdgcn_set_fpenv, "vWUi", "n")
+//===--===//
+
+// Wave Reduction builtins.
+
@@ -20212,6 +20212,59 @@ Value *CodeGenFunction::EmitAMDGPUBuiltinExpr(unsigned
BuiltinID,
llvm::Value *Env = EmitScalarExpr(E->getArg(0));
return Builder.CreateCall(F, {Env});
}
+ case AMDGPU::BI__builtin_amdgcn_wave_reduce_add_i32:
+ case AMDGPU::BI__builtin_amdg
@@ -202,6 +202,7 @@
// CHECK-NEXT: xqcilo 0.2 'Xqcilo' (Qualcomm uC Large
Offset Load Store Extension)
// CHECK-NEXT: xqcilsm 0.2 'Xqcilsm' (Qualcomm uC Load
Store Multiple Extension)
// CHECK-NEXT: xqcisls 0.2
@@ -11,8 +11,8 @@
///
//===--===//
-#ifndef LLVM_CLANG_CIR_DIALECT_IR_CIROPS
-#define LLVM_CLANG_CIR_DIALECT_IR_CIROPS
+#ifndef CLANG_CIR_DIALECT_IR_CIROPS_TD
dkolsen-pgi wrote:
Because the
Author: Timothy Herchen
Date: 2025-02-18T14:55:38-08:00
New Revision: 9bf582fc090f28c2423fdf472635c39145a13bc9
URL:
https://github.com/llvm/llvm-project/commit/9bf582fc090f28c2423fdf472635c39145a13bc9
DIFF:
https://github.com/llvm/llvm-project/commit/9bf582fc090f28c2423fdf472635c39145a13bc9.dif
https://github.com/aheejin closed
https://github.com/llvm/llvm-project/pull/124365
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -202,6 +202,7 @@
// CHECK-NEXT: xqcilo 0.2 'Xqcilo' (Qualcomm uC Large
Offset Load Store Extension)
// CHECK-NEXT: xqcilsm 0.2 'Xqcilsm' (Qualcomm uC Load
Store Multiple Extension)
// CHECK-NEXT: xqcisls 0.2
@@ -368,7 +370,12 @@ void OHOS::addExtraOpts(llvm::opt::ArgStringList &CmdArgs)
const {
CmdArgs.push_back("-z");
CmdArgs.push_back("relro");
CmdArgs.push_back("-z");
- CmdArgs.push_back("max-page-size=4096");
+ // LoongArch needs page size 16K
SixWeini
@@ -368,7 +370,12 @@ void OHOS::addExtraOpts(llvm::opt::ArgStringList &CmdArgs)
const {
CmdArgs.push_back("-z");
CmdArgs.push_back("relro");
CmdArgs.push_back("-z");
- CmdArgs.push_back("max-page-size=4096");
+ // LoongArch needs page size 16K
+ if (getArch() == llvm:
https://github.com/SixWeining edited
https://github.com/llvm/llvm-project/pull/127555
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/SixWeining commented:
LGTM for the LoongArch change. We'll see if @kpdev has any objections later.
https://github.com/llvm/llvm-project/pull/127555
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bi
HaohaiWen wrote:
This tittle looks like you are adding a new option -lto-sample-profile for LLD.
It's recommended to rename the tittle so that other developer can easily know
you are trying to pass profile file to lto backend via -lto-sample-profile.
https://github.com/llvm/llvm-project/pull/12
https://github.com/farzonl approved this pull request.
https://github.com/llvm/llvm-project/pull/127098
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1104,9 +1104,13 @@ void Sema::ActOnStartOfTranslationUnit() {
}
void Sema::ActOnEndOfTranslationUnitFragment(TUFragmentKind Kind) {
- // No explicit actions are required at the end of the global module fragment.
- if (Kind == TUFragmentKind::Global)
+ if (Kind == TUFrag
@@ -741,16 +741,28 @@ std::unique_ptr AllocateTarget(const
llvm::Triple &Triple,
case llvm::Triple::loongarch32:
switch (os) {
case llvm::Triple::Linux:
- return std::make_unique>(Triple,
- Opt
@@ -2458,7 +2458,8 @@ void
Generic_GCC::GCCInstallationDetector::AddDefaultGCCPrefixes(
static const char *const LoongArch64LibDirs[] = {"/lib64", "/lib"};
static const char *const LoongArch64Triples[] = {
- "loongarch64-linux-gnu", "loongarch64-unknown-linux-gnu"};
https://github.com/SixWeining commented:
I think you should add more tests like
`clang/test/Driver/loongarch-toolchain.c` and https://reviews.llvm.org/D55029.
https://github.com/llvm/llvm-project/pull/127555
___
cfe-commits mailing list
cfe-commits@li
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
https://github.com/SixWeining edited
https://github.com/llvm/llvm-project/pull/127555
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/keith created
https://github.com/llvm/llvm-project/pull/127734
This allows you to use `"directory": ".",` in `compile_commands.json`
and have it be understood for file lookup by `clangd`.
>From 6ad2014e4c69c3d949f081b522630f804a4db567 Mon Sep 17 00:00:00 2001
From: Keith Smi
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Keith Smiley (keith)
Changes
This allows you to use `"directory": ".",` in `compile_commands.json`
and have it be understood for file lookup by `clangd`.
---
Full diff: https://github.com/llvm/llvm-project/pull/127734.diff
2 Files Affec
@@ -399,6 +399,32 @@ TEST(findCompileArgsInJsonDatabase, FindsEntry) {
EXPECT_EQ("command4", FoundCommand.CommandLine[0]) << ErrorMessage;
}
+TEST(findCompileArgsInJsonDatabase, FindsEntryRelativeDirectory) {
+ StringRef Directory(".");
+ StringRef FileName("file");
+ Str
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
@@ -0,0 +1,203 @@
+//===--===//
+//
+// 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
llvmbot wrote:
@llvm/pr-subscribers-backend-webassembly
Author: Sam Clegg (sbc100)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/127721.diff
6 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+7)
- (modified) clang/lib/Basic/Targets/WebAssembly.cpp (+1)
-
https://github.com/sbc100 created
https://github.com/llvm/llvm-project/pull/127721
None
>From 6d8b255090f08835b8b6b22e7c3d2abfe5ef7531 Mon Sep 17 00:00:00 2001
From: Sam Clegg
Date: Tue, 18 Feb 2025 15:34:13 -0800
Subject: [PATCH] [WebAssembly] Enable extended-const feature by default
---
cl
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Sam Clegg (sbc100)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/127721.diff
6 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+7)
- (modified) clang/lib/Basic/Targets/WebAssembly.cpp (+1)
- (modified) cl
@@ -0,0 +1,39 @@
+//===- HLSLTargetInfo.h
---===//
+//
+// 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/bogner commented:
This is looking pretty good, though I have some misgivings about
`CommonHLSLTargetCodeGenInfo` and think it might be better to introduce a
`HLSLLayoutBuilder` that can be used by targets as necessary instead. If you
take that suggestion a couple of my comme
https://github.com/bogner edited
https://github.com/llvm/llvm-project/pull/124886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
@@ -0,0 +1,39 @@
+//===- HLSLTargetInfo.h
---===//
+//
+// 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
@@ -56,9 +75,18 @@ llvm::Type
*DirectXTargetCodeGenInfo::getHLSLType(CodeGenModule &CGM,
return llvm::TargetExtType::get(Ctx, TypeName, {ElemType}, Ints);
}
- case llvm::dxil::ResourceClass::CBuffer:
-llvm_unreachable("dx.CBuffer handles are not implemented yet");
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
@@ -0,0 +1,39 @@
+//===- HLSLTargetInfo.h
---===//
+//
+// 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,70 @@
+//===--- UnintendedCharOstreamOutputCheck.cpp - clang-tidy
+//-===//
EugeneZelenko wrote:
Should be merged into previous line.
https://github.com/llvm/llvm-project/pull/127720
___
@@ -0,0 +1,39 @@
+//===- HLSLTargetInfo.h
---===//
+//
+// 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,70 @@
+// RUN: %check_clang_tidy %s bugprone-unintended-char-ostream-output %t
+
+namespace std {
+
+template class basic_ostream {
+public:
+ basic_ostream &operator<<(int);
+ basic_ostream &operator<<(unsigned int);
+};
+
+template
+basic_ostream &operator<<(basic
@@ -696,12 +661,12 @@ static void CheckFallThroughForBody(Sema &S, const Decl
*D, const Stmt *Body,
if (CD.checkDiagnostics(Diags, ReturnsVoid, HasNoReturn))
return;
SourceLocation LBrace = Body->getBeginLoc(), RBrace = Body->getEndLoc();
- auto EmitDiag = [&](Sourc
@@ -0,0 +1,203 @@
+//===--===//
+//
+// 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,134 @@
+//===--===//
+//
+// 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
@@ -91,6 +91,12 @@ Improvements to clang-tidy
New checks
^^
+- New :doc:`bugprone-unintended-char-ostream-output
+ ` check.
+
+ Finds unintended character output from `unsigned char` and `signed char` to
an
EugeneZelenko wrote:
```suggestion
Find
@@ -115,48 +83,176 @@ llvm::Triple::ArchType CGHLSLRuntime::getArch() {
return CGM.getTarget().getTriple().getArch();
}
-void CGHLSLRuntime::addConstant(VarDecl *D, Buffer &CB) {
- if (D->getStorageClass() == SC_Static) {
-// For static inside cbuffer, take as global st
jhuber6 wrote:
The fix will need to be backported, but I don't think this fixed it all the way
so we'll need another follow-up.
https://github.com/llvm/llvm-project/pull/127528
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm
@@ -56,9 +75,18 @@ llvm::Type
*DirectXTargetCodeGenInfo::getHLSLType(CodeGenModule &CGM,
return llvm::TargetExtType::get(Ctx, TypeName, {ElemType}, Ints);
}
- case llvm::dxil::ResourceClass::CBuffer:
-llvm_unreachable("dx.CBuffer handles are not implemented yet");
https://github.com/hekota edited
https://github.com/llvm/llvm-project/pull/124886
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
efriedma-quic wrote:
I'm not really happy with that... but if the semantics were never properly
defined in the first place, I guess this isn't making things worse.
Not sure if marking the allocas themselves is actually the right approach
long-term. It seems like there's a distinction between
@@ -87,6 +92,31 @@ class RawPtrRefMemberChecker
}
}
+ void visitObjCDecl(const ObjCContainerDecl *CD) const {
+if (auto *ID = dyn_cast(CD)) {
rniwa wrote:
I don't think so. `ObjCContainerDecl` could be `ObjCCategoryDecl` or
`ObjCProtocolDecl` as w
@@ -87,6 +92,31 @@ class RawPtrRefMemberChecker
}
}
+ void visitObjCDecl(const ObjCContainerDecl *CD) const {
+if (auto *ID = dyn_cast(CD)) {
+ for (auto *Ivar : ID->ivars())
+visitIvarDecl(CD, Ivar);
+ return;
+}
+if (auto *ID = dyn_cast(C
1 - 100 of 537 matches
Mail list logo
