https://github.com/hokein approved this pull request.
nit: add a note in `clang/docs/ReleaseNotes.rst`
https://github.com/llvm/llvm-project/pull/92654
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listi
https://github.com/hokein created
https://github.com/llvm/llvm-project/pull/92721
None
>From 2aa2caf15282d28e9fda99d51776d842d3d57e95 Mon Sep 17 00:00:00 2001
From: Haojian Wu
Date: Sun, 19 May 2024 22:47:14 +0200
Subject: [PATCH] [Sema] Fix an out-of-bounds crash when diagnosing bad
conversi
https://github.com/MaskRay approved this pull request.
https://github.com/llvm/llvm-project/pull/92717
___
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: Haojian Wu (hokein)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/92721.diff
3 Files Affected:
- (modified) clang/docs/ReleaseNotes.rst (+2)
- (modified) clang/lib/Sema/SemaOverload.cpp (+8-2)
- (modified) clang/te
@@ -0,0 +1,106 @@
+//===--- radsan_context.cpp - Realtime Sanitizer --*- 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/ornata approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/91446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/hanickadot approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/91446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nikolas Klauser
Date: 2024-05-20T09:36:45+02:00
New Revision: 6582efc263c0df921b88b03bbdcd28a89daaa641
URL:
https://github.com/llvm/llvm-project/commit/6582efc263c0df921b88b03bbdcd28a89daaa641
DIFF:
https://github.com/llvm/llvm-project/commit/6582efc263c0df921b88b03bbdcd28a89daaa641.dif
https://github.com/philnik777 closed
https://github.com/llvm/llvm-project/pull/86652
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
jh7370 wrote:
Just noting that this is on my radar, but I haven't found time to look at it
properly yet.
https://github.com/llvm/llvm-project/pull/91280
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/lis
Author: Sven van Haastregt
Date: 2024-05-20T09:37:53+02:00
New Revision: da6a0b7af29a222b2e16a10155b49d4fafe967f3
URL:
https://github.com/llvm/llvm-project/commit/da6a0b7af29a222b2e16a10155b49d4fafe967f3
DIFF:
https://github.com/llvm/llvm-project/commit/da6a0b7af29a222b2e16a10155b49d4fafe967f3.
https://github.com/svenvh closed https://github.com/llvm/llvm-project/pull/91950
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/budimirarandjelovicsyrmia updated
https://github.com/llvm/llvm-project/pull/70024
From 9b9177b9b6e42fdf1cd54ccd9f16aae892b59f28 Mon Sep 17 00:00:00 2001
From: budimirarandjelovicsyrmia
Date: Fri, 5 Apr 2024 15:20:37 +0200
Subject: [PATCH] [clang] Catch missing format attribut
nikic wrote:
@MaskRay It looks like the new version still causes large compile-time
regressions for sqlite3 debug builds:
http://llvm-compile-time-tracker.com/compare.php?from=7529fe2e92e79eef22a528a7168e4dd777d6e9bd&to=9500a5d02e23f9b43294e5f662ac099f8989c0e4&stat=instructions:u
It's smaller
Author: Daniel Grumberg
Date: 2024-05-20T08:59:02+01:00
New Revision: 50b2bd4a25bb3a7a0790cb59e1240099efd7092e
URL:
https://github.com/llvm/llvm-project/commit/50b2bd4a25bb3a7a0790cb59e1240099efd7092e
DIFF:
https://github.com/llvm/llvm-project/commit/50b2bd4a25bb3a7a0790cb59e1240099efd7092e.dif
https://github.com/daniel-grumberg closed
https://github.com/llvm/llvm-project/pull/92522
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,55 @@
+// RUN: %clang_cc1 -emit-llvm -triple i386-linux -Wno-unknown-pragmas %s -o -
| FileCheck %s
spavloff wrote:
No, this macro is managed by the code in Sema, because pragma FENV_ROUND is
processed there. If only `-E` is specified, __ROUNDING_MOD
https://github.com/chapuni updated
https://github.com/llvm/llvm-project/pull/91446
>From d3ee2d086d07ce0803117ec9fa16ac72801619c7 Mon Sep 17 00:00:00 2001
From: NAKAMURA Takumi
Date: Thu, 2 May 2024 09:36:39 +0900
Subject: [PATCH 1/2] [Coverage] Rework !SystemHeadersCoverage
- Introduce `LeafE
bd1976bris wrote:
> LGTM. I noted a couple of redundant checks, but if you think it's better for
> them to be explicit, I'm okay with keeping them.
Thanks. I think in this case it makes it more obvious what is primarily been
checked in these cases to have the check explicit.
https://github.co
https://github.com/spavloff ready_for_review
https://github.com/llvm/llvm-project/pull/92699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: bd1976bris
Date: 2024-05-20T09:48:35+01:00
New Revision: 384bf545a1a7d8dfd16afd20ef07eb845495bac4
URL:
https://github.com/llvm/llvm-project/commit/384bf545a1a7d8dfd16afd20ef07eb845495bac4
DIFF:
https://github.com/llvm/llvm-project/commit/384bf545a1a7d8dfd16afd20ef07eb845495bac4.diff
LO
https://github.com/bd1976bris closed
https://github.com/llvm/llvm-project/pull/92091
___
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: Serge Pavlov (spavloff)
Changes
The forthcoming C standard defines pragma FENV_ROUND to support constant
rounding mode. It also requires some functions to be evaluated with such mode,
N3096 7.6.2p4 states:
Within the scope of an FENV
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 e76b257483e6c6743de0fa6eca4d0cc60e08385d
db1933033fd37bbbab0b845eed53405db365b0e6 --
https://github.com/vikramRH edited
https://github.com/llvm/llvm-project/pull/92725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
rsandifo-arm wrote:
> clang specifically diagnoses always_inline functions. So for example, say you
> want to write something like:
>
> ```
> #include
> __attribute__((always_inline, target("+sve")))
> static inline void f(void* p) __arm_streaming_compatible {
> *(svuint32_t*)p = svmul_m(svp
@@ -0,0 +1,29 @@
+// RUN: %clang_cc1 -triple aarch64-none-linux-gnu -target-feature +sme -verify
-emit-llvm %s
momchil-velikov wrote:
Thanks!
https://github.com/llvm/llvm-project/pull/91606
___
cfe-commits mailing lis
Author: NAKAMURA Takumi
Date: 2024-05-20T18:06:03+09:00
New Revision: 702a2b627ff4b2a5d330a7bd0d3f7cadaff0b4ed
URL:
https://github.com/llvm/llvm-project/commit/702a2b627ff4b2a5d330a7bd0d3f7cadaff0b4ed
DIFF:
https://github.com/llvm/llvm-project/commit/702a2b627ff4b2a5d330a7bd0d3f7cadaff0b4ed.dif
https://github.com/chapuni closed
https://github.com/llvm/llvm-project/pull/91446
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ivanmurashko wrote:
Friendly ping
@sam-mccall , @jansvoboda11 , could you look at the diff. It fixes a critical
issue for anyone who wants to use Clang modules with file remapping, for
instance at clangd.
https://github.com/llvm/llvm-project/pull/91645
cor3ntin wrote:
> It was quite extensively discussed in
> https://github.com/llvm/llvm-project/pull/84184, but it wasn't deemed
> necessary to go through RFC process for a non-functional change that doesn't
> affect stable interfaces.
We had buy-in to split languages out and I think that has
https://github.com/nikic closed https://github.com/llvm/llvm-project/pull/89872
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/wzssyqa created
https://github.com/llvm/llvm-project/pull/92728
Commit: d59bc6b5c75384aa0b1e78cc85e17e8acaccebaf
Clang/MIPS: Add +fp64 if MSA and no explicit -mfp option (#91949)
added +fp64 for `clang`, while not for `clang -cc1`. So
clang -cc1 -triple=mips -targe
wzssyqa wrote:
Depens on: https://github.com/llvm/llvm-project/pull/92728
https://github.com/llvm/llvm-project/pull/91968
___
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: YunQiang Su (wzssyqa)
Changes
Commit: d59bc6b5c75384aa0b1e78cc85e17e8acaccebaf
Clang/MIPS: Add +fp64 if MSA and no explicit -mfp option (#91949)
added +fp64 for `clang`, while not for `clang -cc1`. So
clang -cc1 -triple=mips -ta
https://github.com/nga888 updated
https://github.com/llvm/llvm-project/pull/92579
>From d0b7fd1062c106f01da7d536185e1702bc09bd09 Mon Sep 17 00:00:00 2001
From: Andrew Ng
Date: Mon, 22 Apr 2024 19:29:01 +0100
Subject: [PATCH 1/2] [clang] Fix PS "selective" DLL import/export of vtable &
typeinfo
@@ -1793,6 +1793,36 @@ void ItaniumCXXABI::EmitDestructorCall(CodeGenFunction
&CGF,
ThisTy, VTT, VTTTy, nullptr);
}
+// Check if any non-inline method has the specified attribute.
+template
+static bool CXXRecordNonInlineHasAttr(const CXXRecordDec
@@ -0,0 +1,115 @@
+/// For a class that has a vtable and typeinfo symbol for RTTI, if a user marks
+/// either:
+///
+/// (a) The entire class as dllexport (dllimport)
+/// (b) Any non-inline method of the class as dllexport (dllimport)
+///
+/// then Clang must export the vta
https://github.com/jtb20 created https://github.com/llvm/llvm-project/pull/92731
This is a minimal patch to support parsing for "omp assume" directives. These
are meant to be hints to a compiler' optimisers: as such, it is legitimate (if
not very useful) to ignore them. The patch builds on top
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
https://github.com/jtb20 edited https://github.com/llvm/llvm-project/pull/92731
___
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: None (jtb20)
Changes
This is a minimal patch to support parsing for "omp assume" directives. These
are meant to be hints to a compiler's optimisers: as such, it is legitimate (if
not very useful) to ignore them. The patch builds on top o
jtb20 wrote:
Adding @jdoerfert as potential reviewer
https://github.com/llvm/llvm-project/pull/92731
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/chapuni updated
https://github.com/llvm/llvm-project/pull/89572
>From 13035b230fd51422f6c3223fcffab4f44bd00956 Mon Sep 17 00:00:00 2001
From: NAKAMURA Takumi
Date: Fri, 19 Apr 2024 15:26:34 +0900
Subject: [PATCH 1/4] [MC/DC][Coverage] Add assertions into emitSourceRegions()
https://github.com/vikramRH edited
https://github.com/llvm/llvm-project/pull/92725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/mstorsjo created
https://github.com/llvm/llvm-project/pull/92737
On Windows, long is always 32 bit, thus one can't use long for casting pointers
to integers, on 64 bit architectures.
Instead use long long, which should be large enough.
This avoids errors like "error: cast f
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Martin Storsjö (mstorsjo)
Changes
On Windows, long is always 32 bit, thus one can't use long for casting pointers
to integers, on 64 bit architectures.
Instead use long long, which should be large enough.
This avoids errors like "error:
https://github.com/MitalAshok updated
https://github.com/llvm/llvm-project/pull/91895
>From 56aed689dc5825fc5bacc6dfdff58ee0eaf71f82 Mon Sep 17 00:00:00 2001
From: Mital Ashok
Date: Sun, 12 May 2024 19:48:24 +0100
Subject: [PATCH 1/8] [Clang] Add attribute for consteval builtins; Declare
const
https://github.com/yronglin edited
https://github.com/llvm/llvm-project/pull/92527
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Lukacma updated
https://github.com/llvm/llvm-project/pull/90849
>From 18c489682d38837d7b0abc7b9ecf829a9df4bd0c Mon Sep 17 00:00:00 2001
From: Marian Lukac
Date: Thu, 2 May 2024 11:33:13 +
Subject: [PATCH 1/3] [Clang] Added lifetime markers for temp. allocated
Aggregate t
https://github.com/robincaloudis created
https://github.com/llvm/llvm-project/pull/92741
Closes https://github.com/llvm/llvm-project/issues/92688
>From 9e8c360029fb6789360ad4296e2f14098db76dd6 Mon Sep 17 00:00:00 2001
From: Robin Caloudis
Date: Mon, 20 May 2024 13:21:32 +0200
Subject: [PATCH 1
@@ -1,13 +1,15 @@
-// RUN: %clang_cc1 -emit-llvm -triple arm64-- -o - %s -O0 | FileCheck %s
--check-prefix=CHECK-O0
-// RUN: %clang_cc1 -emit-llvm -disable-llvm-optzns -triple arm64-- -o - %s -O3
| FileCheck %s --check-prefix=CHECK-O3
+// NOTE: Assertions have been autogenerated
https://github.com/vikramRH edited
https://github.com/llvm/llvm-project/pull/92725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/92740
>From d306d06f7f5455788016f985b44fdf9672c90b86 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Mon, 20 May 2024 14:22:20 +0300
Subject: [PATCH 1/2] Disable compiling Flang and running its tests on Clang
robincaloudis wrote:
Even though this PR gives us the correct behavior, I do not think it is good.
Since I am by no means an expert on Clang, a few questions arose
* Why is `xor` tokenized as unary operator even though it's a word token in C?
* How to properly distinguish between keyword differ
https://github.com/vikramRH edited
https://github.com/llvm/llvm-project/pull/92725
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
> What is the goal here?
The eventual goal is for Sema to have appropriate layering instead of being a
monolithic design. "Appropriate layering" can be handled in many different
ways, but the goal when this was discussed extensively in the past was to split
by "category" w
@@ -190,6 +190,16 @@ class SourceMappingRegion {
bool isBranch() const { return FalseCount.has_value(); }
+ bool isMCDCBranch() const {
+const auto *BranchParams =
std::get_if(&MCDCParams);
+assert(BranchParams == nullptr || BranchParams->ID >= 0);
+return (Br
Endilll wrote:
https://buildkite.com/llvm-project/github-pull-requests/builds/65498#018f95c2-a52b-4d95-97ac-ea7e0c7be9ae
confirms that Flang is no longer compiler, and Flang tests are no longer
executed.
https://github.com/llvm/llvm-project/pull/92740
__
https://github.com/Endilll ready_for_review
https://github.com/llvm/llvm-project/pull/92740
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyctree created
https://github.com/llvm/llvm-project/pull/92743
Fixes https://github.com/clangd/clangd/issues/2042
>From 6b1a52b17559f2d7a93b1fc95207e5706039956f Mon Sep 17 00:00:00 2001
From: zyctree
Date: Mon, 20 May 2024 20:12:56 +0800
Subject: [PATCH] [clang][CodeComplet
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
@llvm/pr-subscribers-clangd
Author: zyctree (zyctree)
Changes
Fixes https://github.com/clangd/clangd/issues/2042
---
Full diff: https://github.com/llvm/llvm-project/pull/92743.diff
2 Files Affected:
- (modified) clang-tools-extra/clangd/unittes
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: zyctree (zyctree)
Changes
Fixes https://github.com/clangd/clangd/issues/2042
---
Full diff: https://github.com/llvm/llvm-project/pull/92743.diff
2 Files Affected:
- (modified) clang-tools-extra/clangd/unittests/CodeCompleteT
@@ -22,6 +22,14 @@
// RUN: %clang -target aarch64 -march=armv9.5-a+cpa -### -c %s 2>&1 |
FileCheck -check-prefix=V95A-CPA %s
// V95A-CPA: "-cc1"{{.*}} "-triple" "aarch64{{.*}}" "-target-cpu" "generic"
"-target-feature" "+v9.5a"{{.*}} "-target-feature" "+cpa"
+// RUN: %clang
https://github.com/MitalAshok edited
https://github.com/llvm/llvm-project/pull/91895
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MitalAshok updated
https://github.com/llvm/llvm-project/pull/91895
>From 56aed689dc5825fc5bacc6dfdff58ee0eaf71f82 Mon Sep 17 00:00:00 2001
From: Mital Ashok
Date: Sun, 12 May 2024 19:48:24 +0100
Subject: [PATCH 1/9] [Clang] Add attribute for consteval builtins; Declare
const
https://github.com/MitalAshok ready_for_review
https://github.com/llvm/llvm-project/pull/91895
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -56,43 +52,64 @@ class Extension<
// The FMV priority
int FMVPriority = _FMVPriority;
+
+// Indicates if the extension is available on the command line.
+string IsFMVOnly = _IsFMVOnly;
}
// Some extensions are available for FMV but can not be controlled
@@ -407,6 +406,16 @@ void IncrementalParser::CleanUpPTU(PartialTranslationUnit
&PTU) {
}
}
}
+
+ // FIXME: We should de-allocate MostRecentTU
+ for (Decl *D : MostRecentTU->decls()) {
+if (!isa(D))
+ continue;
+// Check if we need to clean up the IdR
@@ -2282,7 +2282,8 @@ void Sema::ActOnPopScope(SourceLocation Loc, Scope *S) {
// Remove this name from our lexical scope, and warn on it if we haven't
// already.
-IdResolver.RemoveDecl(D);
+if (!PP.isIncrementalProcessingEnabled())
+ IdResolver.RemoveDec
@@ -2157,11 +2157,11 @@ void ASTStmtWriter::VisitSizeOfPackExpr(SizeOfPackExpr
*E) {
void ASTStmtWriter::VisitPackIndexingExpr(PackIndexingExpr *E) {
VisitExpr(E);
Record.push_back(E->TransformedExpressions);
+ Record.push_back(E->EmptyPack);
Record.AddSourceLocation(E
@@ -4377,15 +4377,20 @@ class PackIndexingExpr final
// The pack being indexed, followed by the index
Stmt *SubExprs[2];
- size_t TransformedExpressions;
+ // The size of the trailing expressions.
+ unsigned TransformedExpressions : 31;
+
+ LLVM_PREFERRED_TYPE(bool)
+
https://github.com/cor3ntin approved this pull request.
https://github.com/llvm/llvm-project/pull/92682
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -68,7 +68,7 @@ function compute-projects-to-test() {
done
;;
clang)
- for p in clang-tools-extra compiler-rt flang lldb cross-project-tests; do
+ for p in clang-tools-extra compiler-rt lldb cross-project-tests; do
tstellar wrote:
Don
@@ -72,7 +72,7 @@ class PrintFunctionsConsumer : public ASTConsumer {
*sema.LateParsedTemplateMap.find(FD)->second;
sema.LateTemplateParser(sema.OpaqueParser, LPT);
llvm::errs() << "late-parsed-decl: \"" << FD->getNameAsString() <<
"\"\n";
-}
+
Endilll wrote:
> Given the concerns raised here, it might make sense to put this change on
> hold, do a few more similar experiments for other parts of C++, and see what
> sort of principled design approach makes sense to take here.
It's not clear to me what do we want to clarify or try out be
@@ -72,7 +72,7 @@ class PrintFunctionsConsumer : public ASTConsumer {
*sema.LateParsedTemplateMap.find(FD)->second;
sema.LateTemplateParser(sema.OpaqueParser, LPT);
llvm::errs() << "late-parsed-decl: \"" << FD->getNameAsString() <<
"\"\n";
-}
+
@@ -68,7 +68,7 @@ function compute-projects-to-test() {
done
;;
clang)
- for p in clang-tools-extra compiler-rt flang lldb cross-project-tests; do
+ for p in clang-tools-extra compiler-rt lldb cross-project-tests; do
Endilll wrote:
The
https://github.com/kito-cheng approved this pull request.
LGTM as the original author of `SemaRISCVVectorLookup.cpp` :)
It's great to see we can put all RISC-V related stuff within same place rather
than many different files.
https://github.com/llvm/llvm-project/pull/92682
erichkeane wrote:
FWIW, I've been working actively on Open ACC Sema, and I found that the split
so far has made development much more organized/easier, and makes naming of
things a lot less awkward. I realize splitting out actual language features is
concerning to some, but I'm cautiously opt
Endilll wrote:
Windows CI has finished with an unrelated failure.
https://buildkite.com/llvm-project/github-pull-requests/builds/65498 confirms
that this patch is working as intended, i.e. Flang is not tested on Clang
changes on both Linux and Windows.
https://github.com/llvm/llvm-project/pul
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/92740
>From d306d06f7f5455788016f985b44fdf9672c90b86 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Mon, 20 May 2024 14:22:20 +0300
Subject: [PATCH 1/3] Disable compiling Flang and running its tests on Clang
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/92740
>From d306d06f7f5455788016f985b44fdf9672c90b86 Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Mon, 20 May 2024 14:22:20 +0300
Subject: [PATCH 1/3] Disable compiling Flang and running its tests on Clang
https://github.com/erichkeane commented:
Hmm... I'm not sure this meets our requirements for inclusion as an attribute.
The semantics of this are pretty opaque, no obvious significant
motivation/applicability in the base languages, etc. There doesn't seem to be
any reasonable use case that I
Author: Haojian Wu
Date: 2024-05-20T16:11:15+02:00
New Revision: 1553b21f6d3b620b8e32121b974793342820ab8c
URL:
https://github.com/llvm/llvm-project/commit/1553b21f6d3b620b8e32121b974793342820ab8c
DIFF:
https://github.com/llvm/llvm-project/commit/1553b21f6d3b620b8e32121b974793342820ab8c.diff
LO
https://github.com/hokein closed https://github.com/llvm/llvm-project/pull/92535
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm edited https://github.com/llvm/llvm-project/pull/89217
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -5387,6 +5387,192 @@ bool
AMDGPULegalizerInfo::legalizeDSAtomicFPIntrinsic(LegalizerHelper &Helper,
return true;
}
+bool AMDGPULegalizerInfo::legalizeLaneOp(LegalizerHelper &Helper,
+ MachineInstr &MI,
+
@@ -5387,6 +5387,192 @@ bool
AMDGPULegalizerInfo::legalizeDSAtomicFPIntrinsic(LegalizerHelper &Helper,
return true;
}
+bool AMDGPULegalizerInfo::legalizeLaneOp(LegalizerHelper &Helper,
+ MachineInstr &MI,
+
@@ -6086,6 +6086,62 @@ static SDValue lowerBALLOTIntrinsic(const
SITargetLowering &TLI, SDNode *N,
DAG.getConstant(0, SL, MVT::i32), DAG.getCondCode(ISD::SETNE));
}
+static SDValue lowerLaneOp(const SITargetLowering &TLI, SDNode *N,
+ Selection
https://github.com/arsenm requested changes to this pull request.
There should be no need to introduce same-sized value casts, whether bitcast or
ptrtoint in either legalizer
https://github.com/llvm/llvm-project/pull/89217
___
cfe-commits mailing lis
@@ -5387,6 +5387,192 @@ bool
AMDGPULegalizerInfo::legalizeDSAtomicFPIntrinsic(LegalizerHelper &Helper,
return true;
}
+bool AMDGPULegalizerInfo::legalizeLaneOp(LegalizerHelper &Helper,
+ MachineInstr &MI,
+
@@ -5387,6 +5387,192 @@ bool
AMDGPULegalizerInfo::legalizeDSAtomicFPIntrinsic(LegalizerHelper &Helper,
return true;
}
+bool AMDGPULegalizerInfo::legalizeLaneOp(LegalizerHelper &Helper,
+ MachineInstr &MI,
+
@@ -407,6 +406,16 @@ void IncrementalParser::CleanUpPTU(PartialTranslationUnit
&PTU) {
}
}
}
+
+ // FIXME: We should de-allocate MostRecentTU
+ for (Decl *D : MostRecentTU->decls()) {
+if (!isa(D))
+ continue;
+// Check if we need to clean up the IdR
@@ -9203,15 +9203,15 @@ static NamedDecl *DiagnoseInvalidRedeclaration(
<< Idx << FDParam->getType()
<< NewFD->getParamDecl(Idx - 1)->getType();
} else if (FDisConst != NewFDisConst) {
- SemaRef.Diag(FD->getLocation(), diag::note_member_def_close_const_
erichkeane wrote:
> @erichkeane Can you please take a look?
Looking now, I was out on Friday due to illness.
https://github.com/llvm/llvm-project/pull/87933
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
@@ -407,6 +406,16 @@ void IncrementalParser::CleanUpPTU(PartialTranslationUnit
&PTU) {
}
}
}
+
+ // FIXME: We should de-allocate MostRecentTU
+ for (Decl *D : MostRecentTU->decls()) {
+if (!isa(D))
+ continue;
+// Check if we need to clean up the IdR
https://github.com/nga888 updated
https://github.com/llvm/llvm-project/pull/92549
>From 98f61948447ecbcbf538c247a274401941a87cd8 Mon Sep 17 00:00:00 2001
From: Andrew Ng
Date: Wed, 15 May 2024 10:55:09 +0100
Subject: [PATCH] [clang] Make PS template DLL attribute propagation the same
as MSVC
https://github.com/erichkeane commented:
Not the best one to be reviewing this, but the code generally looks good to me.
https://github.com/llvm/llvm-project/pull/92527
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-
1 - 100 of 427 matches
Mail list logo