Alejandro =?utf-8?q?Álvarez_Ayllón?=,
Alejandro =?utf-8?q?Álvarez_Ayllón?Message-ID:
In-Reply-To:
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/108021
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists
Author: Alejandro Álvarez Ayllón
Date: 2024-09-18T10:11:03+02:00
New Revision: 0dd56858fe188419182a57d0e03c8cd0aa693867
URL:
https://github.com/llvm/llvm-project/commit/0dd56858fe188419182a57d0e03c8cd0aa693867
DIFF:
https://github.com/llvm/llvm-project/commit/0dd56858fe188419182a57d0e03c8cd0aa6
@@ -0,0 +1,95 @@
+// RUN: %clang_cc1 -verify -triple x86_64-unknown-linux-gnu -fsyntax-only
--embed-dir=%S/Inputs -std=c23 %s -pedantic -Wall
+
+#include
+
+enum us : unsigned short {
+ us_max = USHRT_MAX,
+ us_violation, // expected-error {{enumerator value 65536 is not
rep
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Alejandro Álvarez Ayllón (alejandro-alvarez-sonarsource)
Changes
Add a new flag `AlwaysSuppressInlineNamespace` to `PrintingPolicy` that is
explicit
about *always* removing inline namespaces regardless of ambiguity.
Specializing a templat
https://github.com/AaronBallman approved this pull request.
LG!
https://github.com/llvm/llvm-project/pull/107260
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,95 @@
+// RUN: %clang_cc1 -verify -triple x86_64-unknown-linux-gnu -fsyntax-only
--embed-dir=%S/Inputs -std=c23 %s -pedantic -Wall
+
+#include
+
+enum us : unsigned short {
+ us_max = USHRT_MAX,
+ us_violation, // expected-error {{enumerator value 65536 is not
rep
https://github.com/Fznamznon closed
https://github.com/llvm/llvm-project/pull/107260
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MK-Alias wrote:
Here a quick summery of the most important things.
The reason I've implemented a `UnsetDefault` is because the program flow
doesn't work as [you
specify](https://github.com/llvm/llvm-project/pull/108005#discussion_r1764069398).
I needed a **untouched** state. Granted that `std
https://github.com/alejandro-alvarez-sonarsource created
https://github.com/llvm/llvm-project/pull/109147
Add a new flag `AlwaysSuppressInlineNamespace` to `PrintingPolicy` that is
explicit
about *always* removing inline namespaces regardless of ambiguity.
Specializing a template from an inlin
@@ -6,6 +6,7 @@
//
//===--===//
+#include "../Config.h"
MK-Alias wrote:
>This can be included without the ../
Yes, but it of-course shouldn't! Confirm the
[standard](https://www.open-std.
Author: Mariya Podchishchaeva
Date: 2024-09-18T16:05:23+02:00
New Revision: a9d9b0a03daf7ca986182477a0866df525cfceff
URL:
https://github.com/llvm/llvm-project/commit/a9d9b0a03daf7ca986182477a0866df525cfceff
DIFF:
https://github.com/llvm/llvm-project/commit/a9d9b0a03daf7ca986182477a0866df525cfce
https://github.com/AaronBallman approved this pull request.
LGTM thank you for the fix!
https://github.com/llvm/llvm-project/pull/108804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
MK-Alias wrote:
> > As mentions in [this
> > post](https://github.com/llvm/llvm-project/issues/63565#issuecomment-2337396222):
> > these to issues remain within vsocde/vscode-clangd:
> >
> > * None: after completion an error squiggly line will occur on the
> > identifier. Would be slightly ni
@@ -1558,6 +1562,60 @@ class InProcessThinBackend : public ThinBackendProc {
return BackendThreadPool.getMaxConcurrency();
}
};
+
+/// This Backend will run ThinBackend process but throw away all the output
from
+/// the codegen. This class facilitates the first codegen
@@ -215,6 +224,76 @@ void warn(Error E, StringRef Whence) {
}
}
+static std::string getPath(StringRef Dir, unsigned Task) {
+ return (Dir + "/" + llvm::Twine(Task) + ".saved_copy.bc").str();
kyulee-com wrote:
> Will this work on windows? Can we use `llvm::
@@ -164,6 +164,22 @@ publishOutlinedHashTree(std::unique_ptr
HashTree) {
CodeGenData::getInstance().publishOutlinedHashTree(std::move(HashTree));
}
+/// Initialize the two-codegen rounds.
+void initializeTwoCodegenRounds();
+
+/// Save the current module before the first co
@@ -444,7 +444,7 @@ const RawComment *ASTContext::getRawCommentForAnyRedecl(
return CommentlessRedeclChains.lookup(CanonicalD);
}();
- for (const auto Redecl : D->redecls()) {
+ for (const auto Redecl : CanonicalD->redecls()) {
HighCommander4 wrote:
>
@@ -444,7 +444,7 @@ const RawComment *ASTContext::getRawCommentForAnyRedecl(
return CommentlessRedeclChains.lookup(CanonicalD);
}();
- for (const auto Redecl : D->redecls()) {
+ for (const auto Redecl : CanonicalD->redecls()) {
zyn0217 wrote:
Yeah, bu
https://github.com/zyn0217 approved this pull request.
Thanks for working on this! I think this probably needs a release note,
otherwise it looks good on the whole.
Please give other folks some time before merging it, and I invited
@AaronBallman for the second pair of eyes.
https://github.com
@@ -0,0 +1,99 @@
+//===- unittests/AST/RawCommentForDeclTestTest.cpp
+//-===//
+//
+// 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:
@@ -181,16 +181,16 @@ llvm::Expected>
getFoldingRanges(ParsedAST &AST) {
// Related issue: https://github.com/clangd/clangd/issues/310
llvm::Expected>
getFoldingRanges(const std::string &Code, bool LineFoldingOnly) {
nico wrote:
Should we revert 70914aa631561
MainakSil wrote:
If it looks good to you guys, please merge it.
https://github.com/llvm/llvm-project/pull/108804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MainakSil updated
https://github.com/llvm/llvm-project/pull/108804
>From 43e6c22f25f419b64678374dd247eaf8bc28fa57 Mon Sep 17 00:00:00 2001
From: Mainak Sil
Date: Mon, 16 Sep 2024 09:50:33 +0530
Subject: [PATCH 01/13] [clang] Increase VecLib bitfield size to 4 bits in
CodeGen
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Congcong Cai (HerrCai0907)
Changes
Fixes: #109083
---
Full diff: https://github.com/llvm/llvm-project/pull/109159.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp
(+15-10)
-
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Congcong Cai (HerrCai0907)
Changes
Fixes: #109083
---
Full diff: https://github.com/llvm/llvm-project/pull/109159.diff
4 Files Affected:
- (modified) clang-tools-extra/clang-tidy/modernize/LoopConvertUtils.cpp
(+15-10)
- (modifi
https://github.com/HerrCai0907 created
https://github.com/llvm/llvm-project/pull/109159
Fixes: #109083
>From 4fc1d24c4ff22a8da22454aebe7053ea76419767 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Wed, 18 Sep 2024 23:26:02 +0800
Subject: [PATCH] [clang-tidy] loop convert can handle lambda
Author: Mainak Sil
Date: 2024-09-18T11:29:49-04:00
New Revision: 475ceca859233b387c22f13ecef581158ef36346
URL:
https://github.com/llvm/llvm-project/commit/475ceca859233b387c22f13ecef581158ef36346
DIFF:
https://github.com/llvm/llvm-project/commit/475ceca859233b387c22f13ecef581158ef36346.diff
LO
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/108804
___
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-x86_64-debian`
running on `lldb-x86_64-debian` while building `clang` at step 3
"cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/162/builds/6566
Here is the relevant piece of t
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`libc-x86_64-debian-dbg-runtimes-build` running on `libc-x86_64-debian` while
building `clang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/78/builds/6198
Here is the relevant
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`libc-aarch64-ubuntu-fullbuild-dbg` running on `libc-aarch64-ubuntu` while
building `clang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/71/builds/6729
Here is the relevant pi
https://github.com/RKSimon created
https://github.com/llvm/llvm-project/pull/109160
Now that we have the C++ `__builtin_elementwise_popcount` intrinsic (#108121) -
replace custom target intrinsics that just immediately map to Intrinsic::ctpop
so use the generic intrinsic for simplicity.
>From
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`libc-x86_64-debian-fullbuild-dbg` running on `libc-x86_64-debian-fullbuild`
while building `clang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/179/builds/6569
Here is the re
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`libc-x86_64-debian-gcc-fullbuild-dbg` running on
`libc-x86_64-debian-fullbuild` while building `clang` at step 4 "annotate".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/131/builds/6662
Here is th
https://github.com/erichkeane approved this pull request.
I think the release note should be mroe detailed and give some help as to the
WHY here, and not just 'we removed it'.
https://github.com/llvm/llvm-project/pull/109154
___
cfe-commits mailing li
@@ -43,6 +43,8 @@ infrastructure are described first, followed by tool-specific
sections.
Major New Features
--
+- The ``clang-pseudo`` tool has been removed.
erichkeane wrote:
A link to the RFC might be nice here, lets folks see the WHY.
ht
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 40c45b6b43180221acb49f387e7d3158adf49e3e
294b2ea315f1cbe1e794c2fb609455e9466e3b35 --e
https://github.com/erichkeane edited
https://github.com/llvm/llvm-project/pull/109154
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MainakSil updated
https://github.com/llvm/llvm-project/pull/108804
>From 43e6c22f25f419b64678374dd247eaf8bc28fa57 Mon Sep 17 00:00:00 2001
From: Mainak Sil
Date: Mon, 16 Sep 2024 09:50:33 +0530
Subject: [PATCH 01/11] [clang] Increase VecLib bitfield size to 4 bits in
CodeGen
https://github.com/MainakSil updated
https://github.com/llvm/llvm-project/pull/108804
>From 43e6c22f25f419b64678374dd247eaf8bc28fa57 Mon Sep 17 00:00:00 2001
From: Mainak Sil
Date: Mon, 16 Sep 2024 09:50:33 +0530
Subject: [PATCH 01/10] [clang] Increase VecLib bitfield size to 4 bits in
CodeGen
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-amdgpu-runtime` running on `omp-vega20-0` while building
`clang` at step 4 "configure-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/30/builds/6456
Here is the relevant piece
https://github.com/AaronBallman created
https://github.com/llvm/llvm-project/pull/109161
Reverts llvm/llvm-project#108804
>From 4cabae1e59f75c9be5a1156cc785c93ece49c531 Mon Sep 17 00:00:00 2001
From: Aaron Ballman
Date: Wed, 18 Sep 2024 11:33:44 -0400
Subject: [PATCH] =?UTF-8?q?Revert=20"[clan
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `flang-aarch64-libcxx`
running on `linaro-flang-aarch64-libcxx` while building `clang` at step 4
"cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/89/builds/6624
Here is the relevant
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/109159
___
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: Aaron Ballman (AaronBallman)
Changes
Reverts llvm/llvm-project#108804
---
Full diff: https://github.com/llvm/llvm-project/pull/109161.diff
4 Files Affected:
- (modified) clang/include/clang/Basic/CodeGenOptions.def (+2-12)
- (removed)
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`openmp-offload-sles-build-only` running on `rocm-worker-hw-04-sles` while
building `clang` at step 4 "configure-openmp".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/140/builds/6859
Here is the re
Author: Aaron Ballman
Date: 2024-09-18T11:34:09-04:00
New Revision: 3f0dfab54184dcf9b00f37a2b8ac4f1f6ab14701
URL:
https://github.com/llvm/llvm-project/commit/3f0dfab54184dcf9b00f37a2b8ac4f1f6ab14701
DIFF:
https://github.com/llvm/llvm-project/commit/3f0dfab54184dcf9b00f37a2b8ac4f1f6ab14701.diff
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/109161
___
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
`llvm-clang-x86_64-win-fast` running on `as-builder-3` while building `clang`
at step 5 "cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/2/builds/7016
Here is the relevant piece of t
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 67518a44fec0f59b2f926059cf15ec77ec72da13
79002b2854e36e911b371a0b738fac0bc0dbb888 --e
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `ppc64le-flang-rhel-clang`
running on `ppc64le-flang-rhel-test` while building `clang` at step 4
"cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/157/builds/8062
Here is the relevant
AaronBallman wrote:
I had to revert due to build breakages (linked in comments):
```
CMake Error at cmake/modules/LLVMProcessSources.cmake:116 (message):
Found erroneous configuration for source file SimulatedOverflowTest.cpp
LLVM's build system enforces that all source files are added to a b
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder
`llvm-clang-x86_64-sie-ubuntu-fast` running on `sie-linux-worker` while
building `clang` at step 4 "cmake-configure".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/144/builds/7339
Here is the releva
https://github.com/steakhal closed
https://github.com/llvm/llvm-project/pull/109104
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Balazs Benics
Date: 2024-09-18T11:37:27+02:00
New Revision: e32a62c0d31cdfd622461eb4758d34adca509a62
URL:
https://github.com/llvm/llvm-project/commit/e32a62c0d31cdfd622461eb4758d34adca509a62
DIFF:
https://github.com/llvm/llvm-project/commit/e32a62c0d31cdfd622461eb4758d34adca509a62.diff
https://github.com/labrinea reopened
https://github.com/llvm/llvm-project/pull/101712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/labrinea updated
https://github.com/llvm/llvm-project/pull/101712
>From 2fe8ef63846989952a1c72b83114aabe6f36637a Mon Sep 17 00:00:00 2001
From: Alexandros Lamprineas
Date: Fri, 2 Aug 2024 17:32:23 +0100
Subject: [PATCH] [AArch64] Split FeatureLS64 to LS64_ACCDATA and LS64_V.
https://github.com/labrinea converted_to_draft
https://github.com/llvm/llvm-project/pull/101712
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 updated
https://github.com/llvm/llvm-project/pull/109068
>From 66be189c281db7a49c5238c2fe09df64842c1e25 Mon Sep 17 00:00:00 2001
From: Congcong Cai
Date: Tue, 17 Sep 2024 23:01:17 +0800
Subject: [PATCH 1/2] add quotation
---
.../modernize/AvoidCArraysCheck.cpp
zmodem wrote:
After this change we're running into #34485 ("ASan strtol interceptor breaks
errno on Windows"):
```
C:\src\chromium\src>type a.cc
#include
#include
int main() {
errno = 0;
long result = strtol("2147483648", nullptr, 10);
printf("errno: %d\n", errno);
printf("result: %l
https://github.com/phoebewang closed
https://github.com/llvm/llvm-project/pull/108537
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/HerrCai0907 edited
https://github.com/llvm/llvm-project/pull/109068
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
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 0dd56858fe188419182a57d0e03c8cd0aa693867
b5e904ed860c27b03f986c89ffbf373a5910abb1 --e
https://github.com/steakhal updated
https://github.com/llvm/llvm-project/pull/109112
>From 408ee82b1ee3ae15302b2a0dde9faded3e43bf0f Mon Sep 17 00:00:00 2001
From: Arseniy Zaostrovnykh
Date: Wed, 18 Sep 2024 11:30:10 +0200
Subject: [PATCH 1/3] [analyzer] Note last "fclose" call from
"ensureStre
https://github.com/steakhal updated
https://github.com/llvm/llvm-project/pull/109112
>From 408ee82b1ee3ae15302b2a0dde9faded3e43bf0f Mon Sep 17 00:00:00 2001
From: Arseniy Zaostrovnykh
Date: Wed, 18 Sep 2024 11:30:10 +0200
Subject: [PATCH 1/4] [analyzer] Note last "fclose" call from
"ensureStre
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/108657
>From 70d1be2a2a0f2f44cdd70bfb4397e7a36f1c9f30 Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sat, 14 Sep 2024 01:46:28 +0300
Subject: [PATCH 1/2] [Clang] prevented assertion failure by handling integral
t
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/107786
>From 8774b4b073b5ebb52bdf35a7c44c613c6fcf56fc Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Fri, 13 Sep 2024 17:55:15 +0300
Subject: [PATCH] [Clang] prevent recovery call expression from proceeding with
https://github.com/MacDue updated
https://github.com/llvm/llvm-project/pull/108853
>From 6db9f6d56f0bbd56d017156f858eae68653fbd1b Mon Sep 17 00:00:00 2001
From: Benjamin Maxwell
Date: Mon, 16 Sep 2024 16:27:23 +
Subject: [PATCH 1/5] Precommit math-libcalls-tbaa-indirect-args.c
---
.../mat
AaronBallman wrote:
> But for clang -verify you have to add checkers for exactly the emitted
> diagnostics. No more or less. So the only real question is on which line to
> place them.
As someone who does *a lot* of review in Clang, I (strongly) disagree with this
assertion.
You do not have
https://github.com/SimplyDanny edited
https://github.com/llvm/llvm-project/pull/106675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,30 @@
+
+External Clang-Tidy Examples
+
+
+Introduction
+
+
+This page provides examples of what people have done with
:program:`clang-tidy` that
+might serve as useful guides (or starting points) to
https://github.com/SimplyDanny approved this pull request.
One more nit. Otherwise looks good to me.
https://github.com/llvm/llvm-project/pull/106675
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinf
https://github.com/SimplyDanny approved this pull request.
https://github.com/llvm/llvm-project/pull/107521
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon ready_for_review
https://github.com/llvm/llvm-project/pull/109160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,202 @@
+//===--- SYCL.cpp - SYCL Tool and ToolChain Implementations -*- 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: Ap
llvmbot wrote:
@llvm/pr-subscribers-backend-powerpc
Author: Simon Pilgrim (RKSimon)
Changes
Now that we have the C++ `__builtin_elementwise_popcount` intrinsic (#108121) - replace custom target intrinsics that just immediately map to
Intrinsic::ctpop so use the generic intrinsic for simpl
https://github.com/RKSimon edited
https://github.com/llvm/llvm-project/pull/109160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/RKSimon edited
https://github.com/llvm/llvm-project/pull/109160
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/shiltian updated
https://github.com/llvm/llvm-project/pull/108786
>From 4d86d8dc50b526ddca3de21709fe78ddafa7280e Mon Sep 17 00:00:00 2001
From: Shilei Tian
Date: Tue, 17 Sep 2024 21:47:45 -0400
Subject: [PATCH] [IR] Introduce `T` to `DataLayout` to
represent flat address spa
@@ -3158,6 +3161,33 @@ checkBuiltinTemplateIdType(Sema &SemaRef,
BuiltinTemplateDecl *BTD,
int64_t N = Index.getExtValue();
return Ts.getPackAsArray()[N].getAsType();
}
+ case BTK__type_pack_dedup: {
+assert(Converted.size() == 2 && "__builtin_type_pack_dedup sh
@@ -189,19 +190,23 @@
#define LLVM_TEMPLATE_ABI __declspec(dllimport)
#define LLVM_EXPORT_TEMPLATE
#endif
+#define LLVM_ABI_EXPORT __declspec(dllexport)
fsfod wrote:
Its macro I mentioned earlier initial explanation about being used in Clang and
LLVM, it shou
@@ -29,15 +29,21 @@ RWBuffer Buffer;
// CHECK-NEXT: CXXRecordDecl 0x{{[0-9A-Fa-f]+}} <> implicit class RWBuffer definition
// CHECK: FinalAttr 0x{{[0-9A-Fa-f]+}} <> Implicit final
-// CHECK-NEXT: implicit h 'element_type * {{\[\[}}hlsl::resource_class(UAV)]]
{{\[\[}}hlsl::co
https://github.com/tarunprabhu updated
https://github.com/llvm/llvm-project/pull/109165
>From 678fddafe35cec58de50b59354bc82a58bb47534 Mon Sep 17 00:00:00 2001
From: Tarun Prabhu
Date: Wed, 18 Sep 2024 09:49:26 -0600
Subject: [PATCH 1/2] [clang][flang] Support -time in both clang and flang
The
elizabethandrews wrote:
> Instead of removing the test case altogether, does it not suffice to remove
> the SYCL reference / SYCL specific flag, which I interpret as the problem
> being addressed? SYCL per se is not a target, but rather a language?
> Apologies if I'm missing something.
I trie
@@ -309,7 +309,10 @@ enum BuiltinTemplateKind : int {
BTK__make_integer_seq,
/// This names the __type_pack_element BuiltinTemplateDecl.
- BTK__type_pack_element
+ BTK__type_pack_element,
+
+ /// This names the __type_list_dedup BuiltinTemplateDecl.
+ BTK__type_list_de
shiltian wrote:
I like this direction and I think it should be the right way. However, IMHO, I
think it needs discussion (and potentially an RFC).
https://github.com/llvm/llvm-project/pull/109152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://github.com/bogner approved this pull request.
https://github.com/llvm/llvm-project/pull/108902
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AlexVlx wrote:
Instead of removing the test case altogether, does it not suffice to remove
the SYCL reference / SYCL specific flag, which I interpret as the problem being
addressed? SYCL per se is not a target, but rather a language? Apologies if I'm
missing something.
https://github.com/llv
@@ -33,7 +33,8 @@
#include "llvm/Support/Debug.h"
namespace {
AlexVlx wrote:
Done.
https://github.com/llvm/llvm-project/pull/106429
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi
hnrklssn wrote:
This change leads to a crash in the following case:
```
struct S {
};
union U {
struct S s;
int x;
};
void foo() {
union U bar = {};
}
```
`isEmptyRecordForLayout` returns false for `union U` because the recursive call
for `U::x` returns false. This means we call `I
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
Author: David Pagan (ddpagan)
Changes
Added codegen for scope directive, enabled allocate and firstprivate clauses,
and added scope directive LIT test.
Testing
- LIT tests (including new scope test).
- OpenMP scope example test from 5
https://github.com/SimplyDanny approved this pull request.
https://github.com/llvm/llvm-project/pull/109178
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -251,6 +251,24 @@ SPIRV::MemorySemantics::MemorySemantics
getMemSemantics(AtomicOrdering Ord) {
llvm_unreachable(nullptr);
}
+SPIRV::Scope::Scope getMemScope(const LLVMContext &Ctx, SyncScope::ID ID) {
AlexVlx wrote:
I've taken this suggestion, but type
https://github.com/Prabhuk updated
https://github.com/llvm/llvm-project/pull/76838
>From 5a05572d9daf9ce640b54975b4b4e874bd9fcba3 Mon Sep 17 00:00:00 2001
From: prabhukr
Date: Mon, 4 Dec 2023 08:54:14 -0800
Subject: [PATCH 1/4] [UEFI] X86_64 UEFI Clang Driver
Introduce changes necessary for UE
@@ -2419,8 +2419,13 @@ Value *ScalarExprEmitter::VisitCastExpr(CastExpr *CE) {
return Visit(const_cast(E));
case CK_NoOp: {
-return CE->changesVolatileQualification() ? EmitLoadOfLValue(CE)
- : Visit(const_cast(E));
+i
peilin-ye wrote:
Hi @eddyz87, thanks for the review and context!
> ```
> lock *(u64 *)(r1 + 0x0) = r2 release
> lock r2 = *(u64 *)(r1 + 0x0) acquire
> ```
Appending `acquire` and `release` does sound nice to me since it makes the
syntax more similar to LLVM IR (e.g. `store atomic i64 %v, ptr %
mizvekov wrote:
@alexfh Your third test case reduces to:
```C++
template struct InlinedVector {};
template
void PackArgsHelper(InlinedVector, unsigned char);
template
void PackArgsHelper(InlinedVector packed_args, T0,
TRest... args_rest) {
PackArgsHelper(packed_args, a
llvm-ci wrote:
LLVM Buildbot has detected a new failure on builder `clang-aarch64-quick`
running on `linaro-clang-aarch64-quick` while building `clang` at step 5 "ninja
check 1".
Full details are available at:
https://lab.llvm.org/buildbot/#/builders/65/builds/4770
Here is the relevant piec
elizabethandrews wrote:
> > > Instead of removing the test case altogether, does it not suffice to
> > > remove the SYCL reference / SYCL specific flag, which I interpret as the
> > > problem being addressed? SYCL per se is not a target, but rather a
> > > language? Apologies if I'm missing so
@@ -621,6 +642,16 @@ let Predicates = [BPFHasLdsx] in {
def LDD : LOADi64;
+class acquiring_load
+: PatFrag<(ops node:$ptr), (base node:$ptr)> {
+ let IsAtomic = 1;
peilin-ye wrote:
Got it, I'll make it generate acquire and release for weaker types, and
301 - 400 of 410 matches
Mail list logo
