Author: Kazu Hirata
Date: 2023-10-20T23:45:20-07:00
New Revision: af447dde5e723e925b9c8fdc6715b07f0caf6161
URL:
https://github.com/llvm/llvm-project/commit/af447dde5e723e925b9c8fdc6715b07f0caf6161
DIFF:
https://github.com/llvm/llvm-project/commit/af447dde5e723e925b9c8fdc6715b07f0caf6161.diff
L
owenca wrote:
I don't want to "advertise" this new style in the documentation but am willing
to be overruled by the majority.
https://github.com/llvm/llvm-project/pull/69814
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.or
koops updated this revision to Diff 557820.
koops added a comment.
Correcting a git-clang-format error.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123235/new/
https://reviews.llvm.org/D123235
Files:
clang/include/clang/AST/ASTNodeTraverser.h
clang/include/clang/AST/OpenMPClause.
https://github.com/owenca created
https://github.com/llvm/llvm-project/pull/69814
None
>From 78a2661ba7a18ecf8b09be71f6959fbc9fcf70fe Mon Sep 17 00:00:00 2001
From: Owen Pan
Date: Fri, 20 Oct 2023 23:01:38 -0700
Subject: [PATCH] [clang-format] Add a new style for the clang-format source
code
https://github.com/xgupta closed https://github.com/llvm/llvm-project/pull/68435
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Arseny
Date: 2023-10-21T10:40:58+05:30
New Revision: 826c93f96efd8e34781c12cfcc6e7284691f6e8c
URL:
https://github.com/llvm/llvm-project/commit/826c93f96efd8e34781c12cfcc6e7284691f6e8c
DIFF:
https://github.com/llvm/llvm-project/commit/826c93f96efd8e34781c12cfcc6e7284691f6e8c.diff
LOG: [
=?utf-8?q?Bj=C3=B6rn_Sch=C3=A4pers?=
Message-ID:
In-Reply-To:
@@ -3102,7 +3102,9 @@ TEST_F(FormatTestComments, DontAlignNamespaceComments) {
StringRef Input = "namespace A {\n"
" TESTSUITE(B) {\n"
"namespace C {\n"
-
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Brad Smith (brad0)
Changes
- FreeBSD removed big-endian arm with 12.0.
- OpenBSD never had big-endian arm support. I added it just in case, but it has
never been used.
- Remove sparcel bits. It was sprinkled in a few places but it will nev
https://github.com/brad0 created https://github.com/llvm/llvm-project/pull/69809
- FreeBSD removed big-endian arm with 12.0.
- OpenBSD never had big-endian arm support. I added it just in case, but it has
never been used.
- Remove sparcel bits. It was sprinkled in a few places but it will never
@@ -0,0 +1,80 @@
+//===-- State.cpp - OpenMP State & ICV interface - 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: Apach
https://github.com/shiltian edited
https://github.com/llvm/llvm-project/pull/69806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,80 @@
+//===-- State.cpp - OpenMP State & ICV interface - 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: Apach
@@ -0,0 +1,80 @@
+//===-- State.cpp - OpenMP State & ICV interface - 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: Apach
@@ -0,0 +1,80 @@
+//===-- State.cpp - OpenMP State & ICV interface - 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: Apach
@@ -36,6 +46,15 @@ extern "C" {
#pragma omp end declare variant
#ifdef __AMDGCN__
+#pragma omp begin declare variant match(
\
+device = {arch(amdgcn)},
\
+implementation =
@@ -36,6 +46,15 @@ extern "C" {
#pragma omp end declare variant
#ifdef __AMDGCN__
+#pragma omp begin declare variant match(
\
+device = {arch(amdgcn)},
\
+implementation =
https://github.com/jhuber6 commented:
I figured we would just have an internal implementation of `malloc` provided by
the OpenMP device runtime library. The variants make this more difficult, but
we could just use `#ifdef __AMDGPU__` instead of an OpenMP variant so we can
declare it as `extern
@@ -0,0 +1,80 @@
+//===-- State.cpp - OpenMP State & ICV interface - 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: Apach
https://github.com/jhuber6 edited
https://github.com/llvm/llvm-project/pull/69806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jdoerfert updated
https://github.com/llvm/llvm-project/pull/69806
>From f3e7e4008cc2acf9101721efd0b7c736b7d982c0 Mon Sep 17 00:00:00 2001
From: Johannes Doerfert
Date: Sat, 30 Sep 2023 23:37:28 -0700
Subject: [PATCH] [OpenMP] Basic BumpAllocator for (AMD)GPUs
The patch conta
https://github.com/jdoerfert updated
https://github.com/llvm/llvm-project/pull/69806
>From 70619108a402b01c5f1f9d307a8cf21289849d74 Mon Sep 17 00:00:00 2001
From: Johannes Doerfert
Date: Sat, 30 Sep 2023 23:37:28 -0700
Subject: [PATCH] [OpenMP] Basic BumpAllocator for (AMD)GPUs
The patch conta
@@ -15,9 +15,19 @@
#endif
#ifdef __cplusplus
+#include
jdoerfert wrote:
this is not freestanding
https://github.com/llvm/llvm-project/pull/69806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
@@ -15,9 +15,19 @@
#endif
#ifdef __cplusplus
+#include
jhuber6 wrote:
I'm unsure if there's ever a reason to include the C++ wrappers in a
freestanding environment. Without C++, these go straight to the `clang`
resource directly. With the C++ versions they
@@ -48,25 +48,17 @@ namespace {
extern "C" {
[[gnu::weak, gnu::leaf]] void *malloc(uint64_t Size);
[[gnu::weak, gnu::leaf]] void free(void *Ptr);
-}
-///}
+void *llvm_device_malloc(uint64_t Size) { return allocator::alloc(Size); }
+void llvm_device_free(void *Ptr) { allocator
@@ -16,3 +16,4 @@ free
memcmp
printf
__assert_fail
+malloc
jhuber6 wrote:
This is already in the file
https://github.com/llvm/llvm-project/pull/69806
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://l
https://github.com/jdoerfert updated
https://github.com/llvm/llvm-project/pull/69806
>From 50807ba3714b5556e9971f3f009fbd8bea70e15f Mon Sep 17 00:00:00 2001
From: Johannes Doerfert
Date: Sat, 30 Sep 2023 23:37:28 -0700
Subject: [PATCH] [OpenMP] Basic BumpAllocator for (AMD)GPUs
The patch conta
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 1cea309b7e2256746c72f3177e1a5b226bed4f83
d30ff2ef5df686c99184eec7caf175371694cd2d --
https://github.com/jdoerfert updated
https://github.com/llvm/llvm-project/pull/69806
>From d0df9cbcb01528760c45898891e175f915dddbb1 Mon Sep 17 00:00:00 2001
From: Johannes Doerfert
Date: Sat, 30 Sep 2023 23:37:28 -0700
Subject: [PATCH] [OpenMP] Basic BumpAllocator for (AMD)GPUs
The patch conta
llvmbot wrote:
@llvm/pr-subscribers-openmp
Author: Johannes Doerfert (jdoerfert)
Changes
The patch contains a basic BumpAllocator for (AMD)GPUs to allow us to run more
tests. The allocator implements `malloc`, both internally and externally, while
we continue to default to the NVIDIA `ma
https://github.com/jdoerfert created
https://github.com/llvm/llvm-project/pull/69806
The patch contains a basic BumpAllocator for (AMD)GPUs to allow us to run more
tests. The allocator implements `malloc`, both internally and externally, while
we continue to default to the NVIDIA `malloc` when
https://github.com/jcsxky closed https://github.com/llvm/llvm-project/pull/68510
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Qizhi Hu
Date: 2023-10-21T09:39:30+08:00
New Revision: 14bc11a651971bbbdf96565898be2c94fa2b1cf5
URL:
https://github.com/llvm/llvm-project/commit/14bc11a651971bbbdf96565898be2c94fa2b1cf5
DIFF:
https://github.com/llvm/llvm-project/commit/14bc11a651971bbbdf96565898be2c94fa2b1cf5.diff
LOG:
yuanfang-chen wrote:
> You are adding a support for these new flags in `flang-new`, which is
> implemented in terms of `clangDriver` - this scenario (i.e. the two projects
> being out of sync) would be very unlikely at the moment (perhaps even
> impossible). And even if that was desired, you'd
=?utf-8?q?Björn_Schäpers?=
Message-ID:
In-Reply-To:
https://github.com/owenca edited https://github.com/llvm/llvm-project/pull/68743
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yuanfang-chen updated
https://github.com/llvm/llvm-project/pull/68896
>From db72a97acc7fd5a517b5ccd525fcf8a7b714fa29 Mon Sep 17 00:00:00 2001
From: Yuanfang Chen
Date: Sat, 21 Oct 2023 01:18:47 +
Subject: [PATCH] [flang][driver] support -dumpversion and -dumpmachine
Matc
=?utf-8?q?Bj=C3=B6rn_Sch=C3=A4pers?=
Message-ID:
In-Reply-To:
@@ -1118,16 +1121,39 @@ void WhitespaceManager::alignTrailingComments() {
}
}
-// We don't want to align namespace end comments.
-const bool DontAlignThisComment =
-I > 0 && C.NewlinesBe
=?utf-8?q?Björn_Schäpers?=
Message-ID:
In-Reply-To:
@@ -1118,16 +1121,39 @@ void WhitespaceManager::alignTrailingComments() {
}
}
-// We don't want to align namespace end comments.
-const bool DontAlignThisComment =
-I > 0 && C.NewlinesBefore == 0
=?utf-8?q?Bj=C3=B6rn_Sch=C3=A4pers?=
Message-ID:
In-Reply-To:
@@ -1118,16 +1121,39 @@ void WhitespaceManager::alignTrailingComments() {
}
}
-// We don't want to align namespace end comments.
-const bool DontAlignThisComment =
-I > 0 && C.NewlinesBe
https://github.com/ZijunZhaoCCK updated
https://github.com/llvm/llvm-project/pull/65148
>From 02e9afd761228f401df4d9f8dfaaca44ffae0c6e Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 31 Aug 2023 20:08:32 +
Subject: [PATCH 01/16] [libc++] Implement ranges::contains
Differential Revision
@@ -33,14 +33,19 @@ handleFrom(const
ast_matchers::internal::Matcher &IsAHandle,
ast_matchers::internal::Matcher handleFromTemporaryValue(
const ast_matchers::internal::Matcher &IsAHandle) {
+
+ const auto TemporaryExpr =
+ anyOf(cxxBindTemporaryExpr(),
+
@@ -140,11 +140,12 @@ void g28(void) {
typedef short v12i16 __attribute((vector_size(24)));
typedef long double v2f80 __attribute((vector_size(24)));
// CHECK: @g28.a = internal global <1 x i64>
- // CHECK: @g28.b = internal global <12 x i16>
- // CHECK: @g28.c = inte
https://github.com/zygoloid commented:
Thanks, this looks good to me other than a small test coverage (see comment) /
diagnostic wording issue.
(For future reference, please don't force-push to PR branches; that makes it
much harder for your reviewer to see what's changed since their last revi
https://github.com/zygoloid edited
https://github.com/llvm/llvm-project/pull/66894
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -45,10 +45,10 @@ config.substitutions.append(('%{flags}',
'-isysroot {}'.format('@CMAKE_OSX_SYSROOT@') if '@CMAKE_OSX_SYSROOT@' else
''
))
config.substitutions.append(('%{compile_flags}',
-'-nostdinc++ -I %{include} -I %{libcxx}/test/support'
+'@CMAKE_CXX_FLAGS_
arichardson wrote:
Debugging shows that the reason this is failing on the clang-cl bots is that
the CMAKE_CXX_FLAGS_INIT are using the MSVC syntax but we pass
--driver-mode=g++ explicitly:
```
lit: C:\ws\src\libcxx\utils\libcxx\test\format.py:55: note: Command
["'C:/Program Files/LLVM/bin/cla
https://github.com/andykaylor closed
https://github.com/llvm/llvm-project/pull/69797
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/andykaylor created
https://github.com/llvm/llvm-project/pull/69797
None
>From 856a471b40f19f5fcf6aab7591009555b6a3841f Mon Sep 17 00:00:00 2001
From: Andy Kaylor
Date: Fri, 20 Oct 2023 12:33:51 -0700
Subject: [PATCH 1/2] Update SimplifyIndVar.cpp
In SimplifyIndvar::replaceF
@@ -44,9 +46,25 @@ bool hasPublicRefAndDeref(const CXXRecordDecl *R) {
namespace clang {
-std::optional
-isRefCountable(const CXXBaseSpecifier* Base)
-{
+std::optional
+hasPublicRefInBase(const CXXBaseSpecifier *Base) {
+ assert(Base);
+
+ const Type *T = Base->getType().ge
@@ -18,24 +18,26 @@ using namespace clang;
namespace {
-bool hasPublicRefAndDeref(const CXXRecordDecl *R) {
+bool hasPublicRefMethod(const CXXRecordDecl *R) {
assert(R);
assert(R->hasDefinition());
- bool hasRef = false;
- bool hasDeref = false;
for (const CXXMet
@@ -70,29 +88,45 @@ std::optional isRefCountable(const CXXRecordDecl* R)
if (!R)
return std::nullopt;
- if (hasPublicRefAndDeref(R))
+ bool hasRef = hasPublicRefMethod(R);
+ bool hasDeref = hasPublicDerefMethod(R);
+ if (hasRef && hasDeref)
return true;
CXX
mehnadnerd wrote:
I've updated it to add regression tests.
https://github.com/llvm/llvm-project/pull/69685
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mehnadnerd updated
https://github.com/llvm/llvm-project/pull/69685
>From d412fa8018ff4788c3ed8a1c8018c293be20040f Mon Sep 17 00:00:00 2001
From: brs
Date: Thu, 19 Oct 2023 17:16:45 -0500
Subject: [PATCH] [RISCV][MC] MC layer support for the experimental zalasr
extension
---
jansvoboda11 wrote:
> Clang part looks fine. For a clang-tidy part, is there a way to test this
> part ? What changes because we use now a absolute build directory. I'm
> missing some tests for that part.
So the clang-tidy part is necessary to keep the
`clang-tidy/infrastructure/clang-tidy-ru
EricWF wrote:
We also don't want to make our test suite flags beholden to changes CMake
chooses to make.
https://github.com/llvm/llvm-project/pull/67201
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
https://github.com/EricWF edited https://github.com/llvm/llvm-project/pull/67201
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -45,10 +45,10 @@ config.substitutions.append(('%{flags}',
'-isysroot {}'.format('@CMAKE_OSX_SYSROOT@') if '@CMAKE_OSX_SYSROOT@' else
''
))
config.substitutions.append(('%{compile_flags}',
-'-nostdinc++ -I %{include} -I %{libcxx}/test/support'
+'@CMAKE_CXX_FLAGS_
dankm wrote:
Hm. I have "fixup" commits in this branch, should I rebase those, or can we
squash merge as-is?
https://github.com/llvm/llvm-project/pull/65812
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
https://github.com/arichardson updated
https://github.com/llvm/llvm-project/pull/67201
>From 6d650b781080d46ccae02793529cf63b18859f3f Mon Sep 17 00:00:00 2001
From: Alex Richardson
Date: Wed, 6 Sep 2023 11:41:56 -0700
Subject: [PATCH 1/4] [libunwind][libc++][libc++abi] Add cross-compilation
fl
https://github.com/ZijunZhaoCCK updated
https://github.com/llvm/llvm-project/pull/69649
>From 0c57e45f6ee16e43cc6388b1ca6beb88bbc7b925 Mon Sep 17 00:00:00 2001
From: zijunzhao
Date: Thu, 19 Oct 2023 22:23:03 +
Subject: [PATCH 1/3] Update stdckdint.h and make it available in pre-C23
modes.
https://github.com/robozati ready_for_review
https://github.com/llvm/llvm-project/pull/69775
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/robozati updated
https://github.com/llvm/llvm-project/pull/69775
>From c644be1d123769395bed6ea069a45c051e3fc66d Mon Sep 17 00:00:00 2001
From: robozati <139823421+roboz...@users.noreply.github.com>
Date: Fri, 20 Oct 2023 17:23:21 -0300
Subject: [PATCH 1/2] Fix RawStringLiteral
https://github.com/shraiysh closed
https://github.com/llvm/llvm-project/pull/69404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shraiysh closed
https://github.com/llvm/llvm-project/pull/69404
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shraiysh updated
https://github.com/llvm/llvm-project/pull/69404
>From cf9e3e16a8b6f56e76c725b6244154b825cc8cc0 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay
Date: Tue, 17 Oct 2023 19:59:11 -0500
Subject: [PATCH 1/2] [OpenMP][mlir] Add translation for `if` in `omp.teams`
T
https://github.com/shraiysh updated
https://github.com/llvm/llvm-project/pull/69404
>From cf9e3e16a8b6f56e76c725b6244154b825cc8cc0 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay
Date: Tue, 17 Oct 2023 19:59:11 -0500
Subject: [PATCH 1/2] [OpenMP][mlir] Add translation for `if` in `omp.teams`
T
https://github.com/robozati converted_to_draft
https://github.com/llvm/llvm-project/pull/69775
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
robozati wrote:
Tests are randomly failing, let me get what is wrong.
https://github.com/llvm/llvm-project/pull/69775
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
dankm wrote:
No _good_ particular reason. I got hung up on the formatting issues then ran
out of steam, and busy with $job. I just ran clang-format on this change and it
came up clean.
And now that I've done that the only reason I have left is I'm unable to merge
my own changes. Would you min
https://github.com/shraiysh updated
https://github.com/llvm/llvm-project/pull/69404
>From cf9e3e16a8b6f56e76c725b6244154b825cc8cc0 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay
Date: Tue, 17 Oct 2023 19:59:11 -0500
Subject: [PATCH] [OpenMP][mlir] Add translation for `if` in `omp.teams`
This
https://github.com/shraiysh updated
https://github.com/llvm/llvm-project/pull/69404
>From cf9e3e16a8b6f56e76c725b6244154b825cc8cc0 Mon Sep 17 00:00:00 2001
From: Shraiysh Vaishay
Date: Tue, 17 Oct 2023 19:59:11 -0500
Subject: [PATCH] [OpenMP][mlir] Add translation for `if` in `omp.teams`
This
This revision was automatically updated to reflect the committed changes.
Closed by commit rGa4323586fcbb: [-Wunsafe-buffer-usage] Add AST info to the
unclaimed DRE debug notes for… (authored by ziqingluo-90).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm
Author: ziqingluo-90
Date: 2023-10-20T14:27:14-07:00
New Revision: a4323586fcbb20f39f00d5d1bc4a94a1aeea15c4
URL:
https://github.com/llvm/llvm-project/commit/a4323586fcbb20f39f00d5d1bc4a94a1aeea15c4
DIFF:
https://github.com/llvm/llvm-project/commit/a4323586fcbb20f39f00d5d1bc4a94a1aeea15c4.diff
https://github.com/xingxue-ibm edited
https://github.com/llvm/llvm-project/pull/69767
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xingxue-ibm edited
https://github.com/llvm/llvm-project/pull/69767
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/xingxue-ibm edited
https://github.com/llvm/llvm-project/pull/69767
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/robozati updated
https://github.com/llvm/llvm-project/pull/69775
>From c644be1d123769395bed6ea069a45c051e3fc66d Mon Sep 17 00:00:00 2001
From: robozati <139823421+roboz...@users.noreply.github.com>
Date: Fri, 20 Oct 2023 17:23:21 -0300
Subject: [PATCH 1/2] Fix RawStringLiteral
mstorsjo wrote:
> ```c
> #if !defined(LLVM_BUILD_SHARED_LIBS)
> ```
>
> is not great but is not too bad. `-DBUILD_SHARED_LIBS=on` modes are slow to
> execute tests and are not used often for Release builds.
I went ahead and relanded this now, in
538b7ba2aacd6e400ee63c4f9ff1c2543ae69a90, with
Author: Martin Storsjö
Date: 2023-10-20T23:34:28+03:00
New Revision: 538b7ba2aacd6e400ee63c4f9ff1c2543ae69a90
URL:
https://github.com/llvm/llvm-project/commit/538b7ba2aacd6e400ee63c4f9ff1c2543ae69a90
DIFF:
https://github.com/llvm/llvm-project/commit/538b7ba2aacd6e400ee63c4f9ff1c2543ae69a90.diff
https://github.com/robozati edited
https://github.com/llvm/llvm-project/pull/69775
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/robozati created
https://github.com/llvm/llvm-project/pull/69775
The `RawStringLiteral` code action runs both on C and C++ versions prior to
C++11, where this feature is unavailable.
This patch adds a condition to check if the context is running a version equal
or greater t
brad0 wrote:
Ping.
https://github.com/llvm/llvm-project/pull/69134
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/69228
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jrbyrnes commented:
Just have a few questions about implementation details -- at a higher level,
seems like we are trading one heuristic for another w.r.t flagging regions as
ExcessRP -- so I'm curious about the relative performance.
https://github.com/llvm/llvm-project/pull
https://github.com/jrbyrnes commented:
Just have a few questions about implementation details -- at a higher level,
seems like we are trading one heuristic for another w.r.t flagging regions as
ExcessRP -- so I'm curious about the relative performance.
https://github.com/llvm/llvm-project/pull
@@ -894,10 +894,22 @@ void GCNSchedStage::setupNewBlock() {
void GCNSchedStage::finalizeGCNRegion() {
DAG.Regions[RegionIdx] = std::pair(DAG.RegionBegin, DAG.RegionEnd);
- DAG.RescheduleRegions[RegionIdx] = false;
jrbyrnes wrote:
Why was this removed?
htt
@@ -959,16 +970,6 @@ void GCNSchedStage::checkScheduling() {
<< DAG.MinOccupancy << ".\n");
}
- unsigned MaxVGPRs = ST.getMaxNumVGPRs(MF);
- unsigned MaxSGPRs = ST.getMaxNumSGPRs(MF);
- if (PressureAfter.getVGPRNum(false) > MaxVGPRs ||
- Pressur
@@ -1117,16 +1118,23 @@ bool
OccInitialScheduleStage::shouldRevertScheduling(unsigned WavesAfter) {
bool UnclusteredHighRPStage::shouldRevertScheduling(unsigned WavesAfter) {
// If RP is not reduced in the unclustered reschedule stage, revert to the
// old schedule.
- if
@@ -702,7 +702,7 @@ bool UnclusteredHighRPStage::initGCNSchedStage() {
if (!GCNSchedStage::initGCNSchedStage())
return false;
- if (DAG.RegionsWithHighRP.none() && DAG.RegionsWithExcessRP.none())
+ if (DAG.RegionsWithExcessRP.none())
jrbyrnes wrote:
W
https://github.com/jrbyrnes commented:
Just have a few questions about implementation details -- at a higher level,
seems like we are trading one heuristic for another w.r.t flagging regions as
ExcessRP -- so I'm curious about the relative performance.
https://github.com/llvm/llvm-project/pull
https://github.com/ian-twilightcoder closed
https://github.com/llvm/llvm-project/pull/69651
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ian Anderson
Date: 2023-10-20T13:23:34-07:00
New Revision: 09ec0004eee2d9929d25cf519956cc470ffb33dd
URL:
https://github.com/llvm/llvm-project/commit/09ec0004eee2d9929d25cf519956cc470ffb33dd
DIFF:
https://github.com/llvm/llvm-project/commit/09ec0004eee2d9929d25cf519956cc470ffb33dd.diff
https://github.com/owenca approved this pull request.
https://github.com/llvm/llvm-project/pull/69473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
> but I also see "you got what you asked for!" as being a reasonable defense to
> that.
That's my thinking indeed, and the reason why I opposed to Aaron's proposal to
implicitly mark 1-bit bit-fields as `preferred_type(bool)`.
https://github.com/llvm/llvm-project/pull/69104
___
llvmbot wrote:
@llvm/pr-subscribers-libunwind
Author: None (xingxue-ibm)
Changes
This PR adds `static_cast` to the value returned from `getLR()` in the AIX
unwinder to avoid warning if `-Wconversion` is specified for building in 32-bit
mode.
---
Full diff: https://github.com/llvm/llvm-p
https://github.com/xingxue-ibm created
https://github.com/llvm/llvm-project/pull/69767
This PR adds `static_cast` to the value returned from `getLR()` in the AIX
unwinder to avoid warning if `-Wconversion` is specified for building in 32-bit
mode.
>From 2255e2c9a66420bba36bf4ba7f3d8db36d7e4df
EricWF wrote:
> We currently support both ways of building the library (-fexceptions and
> -fno-exceptions). I don't know how widely used the -fno-exceptions variant is
> used and we could discuss dropping it (and forcing people to always build the
> library itself with -fexceptions), but IMO
https://github.com/rmaz updated https://github.com/llvm/llvm-project/pull/68023
>From 5e3c6319c6a98a07dab6571f65ad1320815d76bf Mon Sep 17 00:00:00 2001
From: Richard Howell
Date: Mon, 2 Oct 2023 11:10:52 -0700
Subject: [PATCH] [clang] add module builtin headers relative to resource dir
When inc
@@ -180,8 +180,9 @@ static void appendSubframeworkPaths(Module *Mod,
OptionalFileEntryRef ModuleMap::findHeader(
Module *M, const Module::UnresolvedHeaderDirective &Header,
SmallVectorImpl &RelativePathName, bool &NeedsFramework) {
- // Search for the header file withi
EricWF wrote:
I'm tempted to just do the non-conforming thing here (because -fno-exceptions
is non-conforming), and just assume that when you're building with
-fno-exceptions, you're not trying to replace the throwing operator new, and
you'll be fine if we have the throwing kind call the non-
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/69104
>From 976aa5c8f3d936a15e7123069a49d97ad3bf7a05 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Sun, 15 Oct 2023 13:14:55 +0300
Subject: [PATCH 01/12] [clang] Add clang::debug_info_type attribute
---
cla
1 - 100 of 348 matches
Mail list logo
