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 4fed3d374dfca82d0cb32bb444985ece04438376
c9e4d58d1c2ecdbbbc4a518ee1698901d0b74e0d --
@@ -334,6 +335,15 @@ template<> struct simplify_type {
}
};
+template <> struct GraphTraits {
wenju-he wrote:
I measured llvm-project compile time impact of this change to llvm/IR/User.h on
intel icx 8358 cpu.
Build command:
`
cmake -GNinja -DLLVM_ENABLE_
Author: Noah Goldstein
Date: 2023-11-01T23:50:35-05:00
New Revision: 51abbf98d19cb1b89c6938811f2805bafe4b336e
URL:
https://github.com/llvm/llvm-project/commit/51abbf98d19cb1b89c6938811f2805bafe4b336e
DIFF:
https://github.com/llvm/llvm-project/commit/51abbf98d19cb1b89c6938811f2805bafe4b336e.diff
https://github.com/goldsteinn closed
https://github.com/llvm/llvm-project/pull/67166
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MaskRay updated this revision to Diff 557975.
MaskRay marked an inline comment as done.
MaskRay retitled this revision from "[ItaniumCXXABI] Add
-fassume-nothrow-exception-dtor to assume that an exception object' destructor
is nothrow" to "[ItaniumCXXABI] Add -fassume-nothrow-exception-dtor to as
ChuanqiXu accepted this revision.
ChuanqiXu added a comment.
LGTM. Thanks.
Comment at: clang/lib/Sema/SemaExprCXX.cpp:1110
+!FT->isNothrow())
+ Diag(ThrowLoc, diag::err_throw_object_throwing_dtor) << Ty << 1 << 1;
+ }
It looks like err
https://github.com/nickdesaulniers approved this pull request.
LGTM; might be nice if it's possible to set this for `opt`/`llc` invocations,
but for now I'm just happy to have this in clang.
https://github.com/llvm/llvm-project/pull/70255
___
cfe-comm
@@ -1965,9 +1965,15 @@ bool Sema::buildCoroutineParameterMoves(SourceLocation
Loc) {
if (PD->getType()->isDependentType())
continue;
+// Preserve the referenced state for unused parameter diagnostics.
+bool DeclReferenced = PD->isReferenced();
--
https://github.com/yuxuanchen1997 edited
https://github.com/llvm/llvm-project/pull/70973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yuxuanchen1997 edited
https://github.com/llvm/llvm-project/pull/70973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Chuanqi Xu
Date: 2023-11-02T14:01:17+08:00
New Revision: d76b56fd28582c1cc6663cefa5ae2f8a23492d0a
URL:
https://github.com/llvm/llvm-project/commit/d76b56fd28582c1cc6663cefa5ae2f8a23492d0a
DIFF:
https://github.com/llvm/llvm-project/commit/d76b56fd28582c1cc6663cefa5ae2f8a23492d0a.diff
LO
https://github.com/ChuanqiXu9 edited
https://github.com/llvm/llvm-project/pull/70973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ChuanqiXu9 approved this pull request.
LGTM.
https://github.com/llvm/llvm-project/pull/70973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1965,9 +1965,15 @@ bool Sema::buildCoroutineParameterMoves(SourceLocation
Loc) {
if (PD->getType()->isDependentType())
continue;
+// Preserve the referenced state for unused parameter diagnostics.
+bool DeclReferenced = PD->isReferenced();
--
Author: Yuxuan Chen
Date: 2023-11-02T14:03:47+08:00
New Revision: 858b56e4962749013ded409ff43370b542c8b6cb
URL:
https://github.com/llvm/llvm-project/commit/858b56e4962749013ded409ff43370b542c8b6cb
DIFF:
https://github.com/llvm/llvm-project/commit/858b56e4962749013ded409ff43370b542c8b6cb.diff
L
https://github.com/ChuanqiXu9 closed
https://github.com/llvm/llvm-project/pull/70973
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Weining Lu
Date: 2023-11-02T14:18:19+08:00
New Revision: 749083b91f31f370cf64831d3e7e6215b6d51442
URL:
https://github.com/llvm/llvm-project/commit/749083b91f31f370cf64831d3e7e6215b6d51442
DIFF:
https://github.com/llvm/llvm-project/commit/749083b91f31f370cf64831d3e7e6215b6d51442.diff
LO
dtellenbach wrote:
@ZequanWu this seems to cause issues on macOS:
https://green.lab.llvm.org/green/job/clang-stage1-RA/36184/console
```
Profile-x86_64 :: Darwin/instrprof-debug-info-correlate.c
Profile-x86_64 :: instrprof-darwin-
Profile-x86_64h :: Darwin/instrprof-debug-info-correlate.c
Profil
https://github.com/SixWeining created
https://github.com/llvm/llvm-project/pull/71025
In g++, empty unions are not ignored like empty structs when flattening structs
to examine whether the structs can be passed via FARs in C++. This patch aligns
clang++ with g++.
Fix https://github.com/llvm/l
MaskRay updated this revision to Diff 557976.
MaskRay marked an inline comment as done.
MaskRay added a comment.
remove unused err_ arguments.
fix and test `throw new B`
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D108905/new/
https://reviews.llvm
llvmbot wrote:
@llvm/pr-subscribers-backend-loongarch
Author: Lu Weining (SixWeining)
Changes
In g++, empty unions are not ignored like empty structs when flattening structs
to examine whether the structs can be passed via FARs in C++. This patch aligns
clang++ with g++.
Fix https://git
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Lu Weining (SixWeining)
Changes
In g++, empty unions are not ignored like empty structs when flattening structs
to examine whether the structs can be passed via FARs in C++. This patch aligns
clang++ with g++.
Fix https://github.com/llvm
@@ -7416,3 +7416,46 @@ that ``p->array`` must have at least ``p->count`` number
of elements available:
}];
}
+
+def CodeAlignAttrDocs : Documentation {
+ let Category = DocCatVariable;
+ let Heading = "clang::code_align";
+ let Content = [{
+The ``clang::code_align(N)``
SixWeining wrote:
Hi @xen0n @xry111, please help to review this. Thanks.
https://github.com/llvm/llvm-project/pull/71025
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vikramRH wrote:
@arsenm, The choice of defaults is based on current state of printf. HIP uses
hostcalls and OpenCL uses buffered variant by default. However I'm willing to
make one of the two variants consistent (preferably hostcalls for me). Do note
that this would make all OpenCL printf to
Author: Ben Shi
Date: 2023-11-02T14:41:33+08:00
New Revision: e98f3bfff2c499e76ce8784ea15cedbfb83b33bf
URL:
https://github.com/llvm/llvm-project/commit/e98f3bfff2c499e76ce8784ea15cedbfb83b33bf
DIFF:
https://github.com/llvm/llvm-project/commit/e98f3bfff2c499e76ce8784ea15cedbfb83b33bf.diff
LOG:
https://github.com/benshi001 closed
https://github.com/llvm/llvm-project/pull/70927
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vikramRH updated
https://github.com/llvm/llvm-project/pull/70932
>From 4c0467078b2f38e814569ad351f86129d1c1d5ee Mon Sep 17 00:00:00 2001
From: Vikram
Date: Wed, 4 Oct 2023 05:41:47 -0400
Subject: [PATCH] [WIP][AMDGPU] hostcall printf support for OpenCL
---
clang/include/cla
401 - 428 of 428 matches
Mail list logo
