@@ -6,11 +6,11 @@ typedef vector float3;
RWBuffer Buffer;
// expected-error@+2 {{class template 'RWBuffer' requires template arguments}}
-// expected-note@*:* {{template declaration from hidden source: template
class RWBuffer final}}
+// expected-note@*:* {{template declarat
https://github.com/vgvassilev updated
https://github.com/llvm/llvm-project/pull/88600
>From b43c52e0ab76815e745d474c40944b4de1a929ab Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Sat, 13 Apr 2024 06:39:34 +
Subject: [PATCH] Fix the double space and double attribute printing of the
f
llvmbot wrote:
@llvm/pr-subscribers-clang
@llvm/pr-subscribers-hlsl
Author: Vassil Vassilev (vgvassilev)
Changes
Fixes #56517.
cc: @kimgr
---
Full diff: https://github.com/llvm/llvm-project/pull/88600.diff
4 Files Affected:
- (modified) clang/lib/AST/DeclPrinter.cpp (+18-13)
- (modif
vgvassilev wrote:
> Ah, that's the expected output -- I can't do anything about that :). See
> #56517.
I believe this should fix it: https://github.com/llvm/llvm-project/pull/88600
Can you test?
https://github.com/llvm/llvm-project/pull/87281
___
cf
https://github.com/vgvassilev created
https://github.com/llvm/llvm-project/pull/88600
Fixes #56517.
cc: @kimgr
>From fe2622816265cf4977410d38dfd32d19df8eff5e Mon Sep 17 00:00:00 2001
From: Vassil Vassilev
Date: Sat, 13 Apr 2024 06:39:34 +
Subject: [PATCH] Fix the double space and double a
Author: Vlad Serebrennikov
Date: 2024-04-13T08:40:26+03:00
New Revision: 5b36bf549c8c900d0f01451e02abd821f38d4538
URL:
https://github.com/llvm/llvm-project/commit/5b36bf549c8c900d0f01451e02abd821f38d4538
DIFF:
https://github.com/llvm/llvm-project/commit/5b36bf549c8c900d0f01451e02abd821f38d4538.
https://github.com/ian-twilightcoder closed
https://github.com/llvm/llvm-project/pull/88432
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Ian Anderson
Date: 2024-04-12T22:36:43-07:00
New Revision: 4138367444a2e3431cdfc7ab6e16c9cab5c76ad8
URL:
https://github.com/llvm/llvm-project/commit/4138367444a2e3431cdfc7ab6e16c9cab5c76ad8
DIFF:
https://github.com/llvm/llvm-project/commit/4138367444a2e3431cdfc7ab6e16c9cab5c76ad8.diff
Author: Cyndy Ishida
Date: 2024-04-12T22:05:12-07:00
New Revision: 79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb
URL:
https://github.com/llvm/llvm-project/commit/79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb
DIFF:
https://github.com/llvm/llvm-project/commit/79dca25f4a0fefd47e9e37d9ce47d84dc0b3bedb.diff
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/88473
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Vlad Serebrennikov
Date: 2024-04-13T08:55:09+04:00
New Revision: 6dcb60481323c871556a5a14ed887a5867ece114
URL:
https://github.com/llvm/llvm-project/commit/6dcb60481323c871556a5a14ed887a5867ece114
DIFF:
https://github.com/llvm/llvm-project/commit/6dcb60481323c871556a5a14ed887a5867ece114.
https://github.com/Endilll closed
https://github.com/llvm/llvm-project/pull/88559
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Jim Lin
Date: 2024-04-13T12:39:30+08:00
New Revision: 94b3c192d3468752e6bc2f20957d433bccdb8ff3
URL:
https://github.com/llvm/llvm-project/commit/94b3c192d3468752e6bc2f20957d433bccdb8ff3
DIFF:
https://github.com/llvm/llvm-project/commit/94b3c192d3468752e6bc2f20957d433bccdb8ff3.diff
LOG:
https://github.com/0x59616e closed
https://github.com/llvm/llvm-project/pull/87264
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/87130
>From 23422a0b3af3e070fed5ae86ed0f67acec066c0a Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Fri, 29 Mar 2024 17:48:14 -0700
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
Author: Fangrui Song
Date: 2024-04-12T20:30:15-07:00
New Revision: f22c30063ddcb7f8e8c4fb0496f515be7f9408b8
URL:
https://github.com/llvm/llvm-project/commit/f22c30063ddcb7f8e8c4fb0496f515be7f9408b8
DIFF:
https://github.com/llvm/llvm-project/commit/f22c30063ddcb7f8e8c4fb0496f515be7f9408b8.diff
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/88326
>From 0afe6a50e4faf95769a17a0d2aaf79b272e991b4 Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 12 Apr 2024 10:47:13 -0700
Subject: [PATCH] [llvm][support] Implement tracing virtual file system
---
..
sdkrystian wrote:
The expression type need to be set to `ASTContext::DependentTy` (currently,
it's set to `ASTContext::OverloadTy`) if we find a member of a base class (as
it is for direct members). Should be an easy fix, so I'm thinking of not
reverting and just opening a PR that will address
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/88326
>From 44791e807564e9c6eb489eccdefb1c0f2647a38a Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 12 Apr 2024 10:47:13 -0700
Subject: [PATCH] [llvm][support] Implement tracing virtual file system
---
..
vsapsai wrote:
The bigger idea is that not enabled stats should be negligibly cheap. As for
me, the problem is in that not being true. Let me check how we have expensive
stats all the time.
Extra fixes are good but I'm concerned they help only with these specific stats
while others might stil
delcypher wrote:
@rapidsna @bwendling I've put the experimental feature flag for late parsing in
a separate PR (#88596). I've not updated this PR to be based on that PR yet but
will do soon. I figured the code for experimental feature flag was large enough
that it deserved its own PR to make t
https://github.com/delcypher updated
https://github.com/llvm/llvm-project/pull/88596
>From 554287a724361a510389d7f34f9b57cf434b9657 Mon Sep 17 00:00:00 2001
From: Dan Liew
Date: Fri, 12 Apr 2024 17:36:19 -0700
Subject: [PATCH] [Attributes] Support Attributes being declared as only
supporting l
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Dan Liew (delcypher)
Changes
[Attributes] Support Attributes being declared as only supporting late parsing
when passing an experimental feature flag
This patch changes the `LateParsed` field of `Attr` in `Attr.td` to be an
instantiation
https://github.com/delcypher created
https://github.com/llvm/llvm-project/pull/88596
[Attributes] Support Attributes being declared as only supporting late parsing
when passing an experimental feature flag
This patch changes the `LateParsed` field of `Attr` in `Attr.td` to be an
instantiation
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (ZijunZhaoCCK)
Changes
Make it part of -Wformat-pedantic.
---
Full diff: https://github.com/llvm/llvm-project/pull/88595.diff
3 Files Affected:
- (modified) clang/include/clang/Basic/DiagnosticSemaKinds.td (+2-2)
- (modified) clan
https://github.com/ZijunZhaoCCK created
https://github.com/llvm/llvm-project/pull/88595
Make it part of -Wformat-pedantic.
>From 29d2d19c9dea9e3f818a34c97f95bd7c93b9433b Mon Sep 17 00:00:00 2001
From: Zijun
Date: Sat, 13 Apr 2024 00:53:38 +
Subject: [PATCH] Carving out -Wformat warning abo
@@ -3734,7 +3734,10 @@ void FunctionProtoType::Profile(llvm::FoldingSetNodeID
&ID, QualType Result,
// spec because of the leading 'bool' which unambiguously indicates
// whether the following bool is the EH spec or part of the arguments.
+ // TODO: The effect set is var
@@ -5153,6 +5156,7 @@ bool FunctionEffect::shouldDiagnoseFunctionCall(
// =
void FunctionEffectSet::Profile(llvm::FoldingSetNodeID &ID) const {
+ ID.AddInteger(size());
if (PImpl)
for (const auto &Effect : *PImpl)
ID.AddInteger(llvm::to_underlying(Effect.ki
@@ -4429,6 +4433,210 @@ class FunctionNoProtoType : public FunctionType, public
llvm::FoldingSetNode {
}
};
+//
--
+
+class Decl;
+class CXXMethodDecl;
+class FunctionEffectSet;
+
+/// Represents a
ayermolo wrote:
I think this commit is causing build failure when building clangd in debug mode
with clang built in release mode.
```
Instruction does not dominate all uses!
%K1104 = getelementptr inbounds %"struct.llvm::json::Object::KV", ptr
%arrayinit.begin1100, i32 0, i32 0, !dbg !93928
jyknight wrote:
>> The end result should be that #imported and #pragma once-guarded files are
>> treated the same way as #ifndef-guarded files.
> While I don't necessarily disagree with that goal in principle, it wasn't
> true before this change either.
Well, yes, I know it isn't true yet --
https://github.com/MaskRay updated
https://github.com/llvm/llvm-project/pull/87130
>From 23422a0b3af3e070fed5ae86ed0f67acec066c0a Mon Sep 17 00:00:00 2001
From: Fangrui Song
Date: Fri, 29 Mar 2024 17:48:14 -0700
Subject: [PATCH 1/3] =?UTF-8?q?[=F0=9D=98=80=F0=9D=97=BD=F0=9D=97=BF]=20in?=
=?UTF
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/87130
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Fangrui Song
Date: 2024-04-12T16:44:46-07:00
New Revision: b9bed1f1106e3116e8bea38806c511b57b956929
URL:
https://github.com/llvm/llvm-project/commit/b9bed1f1106e3116e8bea38806c511b57b956929
DIFF:
https://github.com/llvm/llvm-project/commit/b9bed1f1106e3116e8bea38806c511b57b956929.diff
@@ -0,0 +1,47 @@
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c -verify %s
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c++
-verify=expected,cxx %s
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+static int f(void) { return 42; }
+int g(void) __attribut
https://github.com/dpaoliello approved this pull request.
https://github.com/llvm/llvm-project/pull/88572
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/delcypher updated
https://github.com/llvm/llvm-project/pull/87596
>From af735f3216dd5db9dcaf164892f3f573731701ec Mon Sep 17 00:00:00 2001
From: Yeoul Na
Date: Wed, 3 Apr 2024 20:58:46 -0700
Subject: [PATCH 1/3] [BoundsSafety] WIP: Make 'counted_by' work for pointer
fields; l
https://github.com/MaskRay edited
https://github.com/llvm/llvm-project/pull/87130
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,47 @@
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c -verify %s
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c++
-verify=expected,cxx %s
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+static int f(void) { return 42; }
+int g(void) __attribut
@@ -3233,6 +3233,17 @@ Sema::CheckBuiltinFunctionCall(FunctionDecl *FDecl,
unsigned BuiltinID,
if (BuiltinCountZeroBitsGeneric(*this, TheCall))
return ExprError();
break;
+
+ case Builtin::BI__builtin_allow_runtime_check: {
+Expr *Arg = TheCall->getArg(0);
+
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/87568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/87568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -3464,6 +3464,34 @@ Query for this feature with
``__has_builtin(__builtin_trap)``.
``__builtin_arm_trap`` is lowered to the ``llvm.aarch64.break`` builtin, and
then to ``brk #payload``.
+``__builtin_allow_runtime_check``
vitalybuka wrote:
Similar __buil
dwblaikie wrote:
Re: code: Looks about right.
Bit unfortunate to store template parameters in different ways (in the
`extraData` for the alias template, but in the `templateParams` for the
composite types) - but I guess it'd be more invasive to try to represent alias
templates as composite ty
@@ -3464,6 +3464,34 @@ Query for this feature with
``__has_builtin(__builtin_trap)``.
``__builtin_arm_trap`` is lowered to the ``llvm.aarch64.break`` builtin, and
then to ``brk #payload``.
+``__builtin_allow_runtime_check``
vitalybuka wrote:
Not sure how t
@@ -0,0 +1,27 @@
+// RUN: %clang_cc1 -triple %ms_abi_triple -Wunused -x c -verify %s
+// RUN: %clang_cc1 -triple %ms_abi_triple -Wunused -verify=expected,cxx %s
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+static int f(void) { return 42; } // cxx-warning{{unused function 'f'}}
+in
@@ -0,0 +1,47 @@
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c -verify %s
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c++
-verify=expected,cxx %s
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+static int f(void) { return 42; }
+int g(void) __attribut
https://github.com/nickdesaulniers approved this pull request.
LGTM; though I have a strong preference to get bugs on file and linked to
regarding:
- issues with non-itanium mangling
- issues with namespaced identifiers.
https://github.com/llvm/llvm-project/pull/87130
__
@@ -0,0 +1,47 @@
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c -verify %s
+// RUN: %clang_cc1 -triple %itanium_abi_triple -Wunused -x c++
-verify=expected,cxx %s
+
+#ifdef __cplusplus
+extern "C" {
+#endif
+static int f(void) { return 42; }
+int g(void) __attribut
https://github.com/nickdesaulniers edited
https://github.com/llvm/llvm-project/pull/87130
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -7735,7 +7735,8 @@ ExprResult Sema::BuildResolvedCallExpr(Expr *Fn,
NamedDecl *NDecl,
}
if (CXXMethodDecl *Method = dyn_cast_or_null(FDecl))
-if (Method->isImplicitObjectMemberFunction())
+if (!isa(CurContext) &&
erichkeane wrote:
I still dont
llvmbot wrote:
Failed to cherry-pick: 0d2bb7f017f13ceae793fab7d83d3e67e8d8d8f8)
https://github.com/llvm/llvm-project/actions/runs/8668528827
Please manually backport the fix and push it to your github fork. Once this is
done, please create a [pull
request](https://github.com/llvm/llvm-proje
MaskRay wrote:
Ping:)
https://github.com/llvm/llvm-project/pull/87130
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tstellar wrote:
/cherry-pick 0d2bb7f017f13ceae793fab7d83d3e67e8d8d8f8)
https://github.com/llvm/llvm-project/pull/88465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tstellar milestoned
https://github.com/llvm/llvm-project/pull/88465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tstellar closed
https://github.com/llvm/llvm-project/pull/88465
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Tom Stellard
Date: 2024-04-12T15:10:04-07:00
New Revision: 0d2bb7f017f13ceae793fab7d83d3e67e8d8d8f8
URL:
https://github.com/llvm/llvm-project/commit/0d2bb7f017f13ceae793fab7d83d3e67e8d8d8f8
DIFF:
https://github.com/llvm/llvm-project/commit/0d2bb7f017f13ceae793fab7d83d3e67e8d8d8f8.diff
ian-twilightcoder wrote:
> The end result should be that #imported and #pragma once-guarded files are
> treated the same way as #ifndef-guarded files.
While I don't necessarily disagree with that goal in principle, it wasn't true
before this change either. There was already some special casing
https://github.com/dwblaikie edited
https://github.com/llvm/llvm-project/pull/85050
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/dwblaikie approved this pull request.
Give it a go
https://github.com/llvm/llvm-project/pull/85050
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1061,6 +1070,16 @@ CodeGenAction::CreateASTConsumer(CompilerInstance &CI,
StringRef InFile) {
CI.getPreprocessor().addPPCallbacks(std::move(Callbacks));
}
+ if (CI.getFrontendOpts().GenReducedBMI &&
+ !CI.getFrontendOpts().ModuleOutputPath.empty()) {
+std:
jyknight wrote:
> The problem I'm trying to fix is that nobody knows when it's appropriate to
> use #import vs #include
But you haven't really (and I think cannot) fixed that.
> using header guards or #pragma once is very "un-Objective-C".
Yes, this is quite unfortunate. The best answer woul
Author: erichkeane
Date: 2024-04-12T14:48:39-07:00
New Revision: 03c393db1821ef4c362c4c228492c6fb03f8876f
URL:
https://github.com/llvm/llvm-project/commit/03c393db1821ef4c362c4c228492c6fb03f8876f
DIFF:
https://github.com/llvm/llvm-project/commit/03c393db1821ef4c362c4c228492c6fb03f8876f.diff
LO
https://github.com/jansvoboda11 updated
https://github.com/llvm/llvm-project/pull/88326
>From 665fd1a3fab4730851d14bc879c34341cbfd6a5e Mon Sep 17 00:00:00 2001
From: Jan Svoboda
Date: Fri, 12 Apr 2024 10:47:13 -0700
Subject: [PATCH] [llvm][support] Implement tracing virtual file system
---
..
github-actions[bot] wrote:
@artemcm Congratulations on having your first Pull Request (PR) merged into the
LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a build,
https://github.com/jansvoboda11 closed
https://github.com/llvm/llvm-project/pull/88152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Artem Chikin
Date: 2024-04-12T14:46:56-07:00
New Revision: 779ba60417b467a6d2d25101b11711c009694315
URL:
https://github.com/llvm/llvm-project/commit/779ba60417b467a6d2d25101b11711c009694315
DIFF:
https://github.com/llvm/llvm-project/commit/779ba60417b467a6d2d25101b11711c009694315.diff
Author: erichkeane
Date: 2024-04-12T14:44:20-07:00
New Revision: db8e182952573fa9198424a6ed1b1bb1676eae62
URL:
https://github.com/llvm/llvm-project/commit/db8e182952573fa9198424a6ed1b1bb1676eae62
DIFF:
https://github.com/llvm/llvm-project/commit/db8e182952573fa9198424a6ed1b1bb1676eae62.diff
LO
https://github.com/bradenhelmer updated
https://github.com/llvm/llvm-project/pull/87287
>From 4d0290b2dc10705cf229e4c1c3b40dd5155ea61c Mon Sep 17 00:00:00 2001
From: Braden Helmer
Date: Mon, 1 Apr 2024 18:09:59 -0400
Subject: [PATCH] Added move for Names parameter in MangledSymbol ctor
---
cl
Author: erichkeane
Date: 2024-04-12T14:42:06-07:00
New Revision: a472e6473e6921f22427d41c3d29e9a26eb4ab32
URL:
https://github.com/llvm/llvm-project/commit/a472e6473e6921f22427d41c3d29e9a26eb4ab32
DIFF:
https://github.com/llvm/llvm-project/commit/a472e6473e6921f22427d41c3d29e9a26eb4ab32.diff
LO
Author: erichkeane
Date: 2024-04-12T14:42:06-07:00
New Revision: eaa3947106e50d6d837e6681fb55cf92c2bb0f9a
URL:
https://github.com/llvm/llvm-project/commit/eaa3947106e50d6d837e6681fb55cf92c2bb0f9a
DIFF:
https://github.com/llvm/llvm-project/commit/eaa3947106e50d6d837e6681fb55cf92c2bb0f9a.diff
LO
https://github.com/jansvoboda11 approved this pull request.
LGTM, thanks a lot!
https://github.com/llvm/llvm-project/pull/88152
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ian-twilightcoder wrote:
Also this change isn't trying to modify `#include` in any kind of way, _only_
`#import`. So it's quite intentional that your test behaves the same before and
after.
https://github.com/llvm/llvm-project/pull/83660
___
cfe-comm
github-actions[bot] wrote:
@ViToni Congratulations on having your first Pull Request (PR) merged into the
LLVM Project!
Your changes will be combined with recent changes from other authors, then
tested
by our [build bots](https://lab.llvm.org/buildbot/). If there is a problem with
a build,
https://github.com/nickdesaulniers closed
https://github.com/llvm/llvm-project/pull/88565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Victor Toni
Date: 2024-04-12T14:33:35-07:00
New Revision: 900be9013fdc3bab9fce906f8a71e59ecd8873b4
URL:
https://github.com/llvm/llvm-project/commit/900be9013fdc3bab9fce906f8a71e59ecd8873b4
DIFF:
https://github.com/llvm/llvm-project/commit/900be9013fdc3bab9fce906f8a71e59ecd8873b4.diff
L
https://github.com/vitalybuka edited
https://github.com/llvm/llvm-project/pull/87568
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From 1989bbbdb1d575e2e938ec319ad9ad23db6af6fc Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
ian-twilightcoder wrote:
I don't really think it's the same thing. The problem I'm trying to fix is that
nobody knows when it's appropriate to use `#import` vs `#include`, and the
unfortunate convention of Objective-C makes it impossible for header owners to
indicate if they support being incl
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From b7c693011abd785bec0a58fe4b6011a84f1ea329 Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
jyknight wrote:
Oh -- I'd also note that even if this is reverted, ilya-biryukov may want to
continue to investigate the source-location issue -- it's entirely possible
that the correct fix will trigger that same problem!
https://github.com/llvm/llvm-project/pull/83660
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From 1dd15b331795daaf2ac70cbd013f56f2d7677041 Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
https://github.com/bader commented:
> I'll emphasise that this is only a problem for things such as implicitly
> generated globals (e.g. VTables or typeinfo for classes etc.)
I suppose usage of VTables and typeinfo is kind of restricted in GPU
programming models. Right?
Anyway, I think it's a
https://github.com/erichkeane closed
https://github.com/llvm/llvm-project/pull/88411
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Erich Keane
Date: 2024-04-12T14:13:31-07:00
New Revision: daa88364df15683bfa0e58d62cc96d33cd49d34b
URL:
https://github.com/llvm/llvm-project/commit/daa88364df15683bfa0e58d62cc96d33cd49d34b
DIFF:
https://github.com/llvm/llvm-project/commit/daa88364df15683bfa0e58d62cc96d33cd49d34b.diff
L
ViToni wrote:
@nickdesaulniers Please. I wouldn't assume to have any permissions in this repo.
https://github.com/llvm/llvm-project/pull/88565
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-c
jyknight wrote:
I think the bug this change was attempting to fix is actually the same as
#38554 and related bugs -- which, it appears, was not really fixed.
The underlying problem here is that `#pragma once` should should work
identically to ifndef guards, as far as what macros/decls are made
https://github.com/ributzka approved this pull request.
https://github.com/llvm/llvm-project/pull/88432
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
artemcm wrote:
> Clang test now looks good to me. Might be nice to drop
> `InstrumentingInMemoryFilesystem` in favor of the existing
> `InstrumentingFilesystem` (that I added just moments ago, sorry!) wrapped
> around a normal `InMemoryFileSystem`, but I'm happy to do that myself in a
> follo
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From 8164aaf2e93dde4761789e6d574088d1cb0e97fc Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
https://github.com/Artem-B approved this pull request.
LGTM. The changes appear to be mechanical in nature, so `check clang` tests
should be sufficient to verify we've re-connected things correctly.
https://github.com/llvm/llvm-project/pull/88559
___
https://github.com/ian-twilightcoder edited
https://github.com/llvm/llvm-project/pull/88432
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ian-twilightcoder wrote:
Upstream from Apple, followup for
https://github.com/llvm/llvm-project/pull/65996. Allows Apple's Darwin module
to include ptrauth.h without declaring a `use`.
https://github.com/llvm/llvm-project/pull/88432
___
cfe-commits m
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From e010a761a34692e5ca1959fc9022d0950e669664 Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
jansvoboda11 wrote:
Clang test now looks good to me. Might be nice to drop
`InstrumentingInMemoryFilesystem` in favor of the existing
`InstrumentingFilesystem` (that I added just moments ago, sorry!) wrapped
around a normal `InMemoryFileSystem`, but I'm happy to do that myself in a
follow-up
https://github.com/ian-twilightcoder updated
https://github.com/llvm/llvm-project/pull/88432
>From 88da8be7ed10f1ee8e7e992fdd59dce52456b2ce Mon Sep 17 00:00:00 2001
From: Alex Lorenz
Date: Thu, 9 Jul 2020 15:10:49 -0700
Subject: [PATCH] [modules] allow use of ptrauth module from
no_undeclared_
jansvoboda11 wrote:
I assume clang-format was overly eager with this one, as these are just
whitespace changes. Can we undo that?
https://github.com/llvm/llvm-project/pull/88152
___
cfe-commits mailing list
cfe-commi
sdkrystian wrote:
@rupprecht looks like the issue is with non-static members in base classes:
```cpp
struct B
{
int z;
void h(int);
};
template
struct A : B
{
int y;
void g(int);
template
void f(U);
template<>
void f(int x)
{
x;
y;
https://github.com/artemcm updated
https://github.com/llvm/llvm-project/pull/88152
>From 24e869df273b9d75bb4fdf85f4ee8ab2ddbccc2c Mon Sep 17 00:00:00 2001
From: Artem Chikin
Date: Tue, 9 Apr 2024 09:37:09 -0700
Subject: [PATCH] [clang][deps] Overload 'Filesystem::exists' in
'DependencyScanning
AlexVlx wrote:
> The change seems reasonable.
>
> > CodeGen/LLVM will default to AS0 in this case, which produces Globals that
> > end up in the private address space for e.g. OCL, HIPSPV or SYCL.
>
> Can we add a test checking LLVM address space for globals emitted from
> OCL/HIPSPV/SYCL, pl
1 - 100 of 336 matches
Mail list logo