https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/112317
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/jurahul edited
https://github.com/llvm/llvm-project/pull/112317
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -0,0 +1,65 @@
+// RUN: %clang_cc1 -triple aarch64-pc-windows-msvc -emit-llvm -o - %s |
FileCheck %s
DanielKristofKiss wrote:
So far, but other targets could be added here later without too much change to
the test.
https://github.com/llvm/llvm-project/pull/1
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/112318
None
>From a9ded784183d3df06434814af9e0ca32d9ef907f Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Mon, 14 Oct 2024 22:56:07 -0700
Subject: [PATCH] [Clang][TableGen] Use const pointer for various `Init`
poin
https://github.com/jurahul created
https://github.com/llvm/llvm-project/pull/112317
None
>From 6dfa15a4c498855bc8bfe363671b7ee17d55d980 Mon Sep 17 00:00:00 2001
From: Rahul Joshi
Date: Mon, 14 Oct 2024 23:12:24 -0700
Subject: [PATCH] [Clang][TableGen] NeonEmitter
---
clang/utils/TableGen/Neo
@@ -107,6 +107,39 @@ namespace std {
using std::operator""s;
using std::operator""sv;
+namespace default_args {
+ using IntArray = int[];
+ const int *defaultparam1(const int &def1 [[clang::lifetimebound]] = 0); //
#def1
+ const int &defaultparam_array([[clang::lifetimebou
@@ -107,6 +107,39 @@ namespace std {
using std::operator""s;
using std::operator""sv;
+namespace default_args {
+ using IntArray = int[];
+ const int *defaultparam1(const int &def1 [[clang::lifetimebound]] = 0); //
#def1
+ const int &defaultparam_array([[clang::lifetimebou
@@ -107,6 +107,39 @@ namespace std {
using std::operator""s;
using std::operator""sv;
+namespace default_args {
+ using IntArray = int[];
bricknerb wrote:
Is this helpful if this is only used once?
https://github.com/llvm/llvm-project/pull/112047
__
@@ -1383,6 +1394,15 @@ static void checkExprLifetimeImpl(Sema &SemaRef,
<< nextPathEntryRange(Path, I + 1, L);
break;
}
+
+ case IndirectLocalPathEntry::DefaultArg: {
+const auto *DAE = cast(Elem.E);
+SemaRef.Diag(DAE->getParam()->
@@ -1370,7 +1381,7 @@ static void checkExprLifetimeImpl(Sema &SemaRef,
break;
}
- case IndirectLocalPathEntry::LambdaCaptureInit:
+ case IndirectLocalPathEntry::LambdaCaptureInit: {
bricknerb wrote:
Revert this change as it is unrelate
https://github.com/bricknerb deleted
https://github.com/llvm/llvm-project/pull/112047
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -1370,7 +1381,7 @@ static void checkExprLifetimeImpl(Sema &SemaRef,
break;
}
- case IndirectLocalPathEntry::LambdaCaptureInit:
+ case IndirectLocalPathEntry::LambdaCaptureInit: {
bricknerb wrote:
Revert this change as it seems unrel
@@ -1159,7 +1159,9 @@ class ARMOperand : public MCParsedAsmOperand {
if (!isImm()) return false;
const MCConstantExpr *CE = dyn_cast(getImm());
if (!CE) return false;
davemgreen wrote:
Can this check that CE->getActiveBits() > 32?
https://github.c
llvmbot wrote:
@llvm/pr-subscribers-clang-static-analyzer-1
Author: None (T-Gruber)
Changes
The current implementation of MemRegion::getDescriptiveName fails for
FieldRegions whose SuperRegion is an ElementRegion. As outlined below:
```Cpp
struct val_struct { int val; };
extern struct val
https://github.com/T-Gruber created
https://github.com/llvm/llvm-project/pull/112313
The current implementation of MemRegion::getDescriptiveName fails for
FieldRegions whose SuperRegion is an ElementRegion. As outlined below:
```Cpp
struct val_struct { int val; };
extern struct val_struct val_s
carlosgalvezp wrote:
Thank you for the contribution! I've added [NFC] (non functional change) to
the commit message title.
https://github.com/llvm/llvm-project/pull/112283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org
github-actions[bot] wrote:
@thevar1able 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 bu
https://github.com/carlosgalvezp closed
https://github.com/llvm/llvm-project/pull/112283
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Konstantin Bogdanov
Date: 2024-10-15T07:53:08+02:00
New Revision: 57d109c4c93c8cf2d7ff9e5877a48152ed7a617a
URL:
https://github.com/llvm/llvm-project/commit/57d109c4c93c8cf2d7ff9e5877a48152ed7a617a
DIFF:
https://github.com/llvm/llvm-project/commit/57d109c4c93c8cf2d7ff9e5877a48152ed7a617a
https://github.com/bricknerb updated
https://github.com/llvm/llvm-project/pull/111856
>From 786d31e2657964e578cd1fdf2006b0fb3b19fab6 Mon Sep 17 00:00:00 2001
From: Boaz Brickner
Date: Thu, 10 Oct 2024 15:15:23 +
Subject: [PATCH 1/5] [clang] When checking for covariant return types, make
su
github-actions[bot] wrote:
@awson 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, y
Author: awson
Date: 2024-10-15T07:52:02+02:00
New Revision: 37bd788138e1f18d5e327de28a6940da57cdcde8
URL:
https://github.com/llvm/llvm-project/commit/37bd788138e1f18d5e327de28a6940da57cdcde8
DIFF:
https://github.com/llvm/llvm-project/commit/37bd788138e1f18d5e327de28a6940da57cdcde8.diff
LOG: [C
https://github.com/cor3ntin closed
https://github.com/llvm/llvm-project/pull/96464
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/NimishMishra approved this pull request.
LGTM! Thanks for this.
The Windows CI failure seems to be in `Driver/atomic.f90`. Do we disable this
test on Windows (if there is a way to do so)?
https://github.com/llvm/llvm-project/pull/112202
__
https://github.com/tbaederr closed
https://github.com/llvm/llvm-project/pull/112238
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Timm Baeder
Date: 2024-10-15T07:49:55+02:00
New Revision: 51d0e40c2ee1ab25f408f22e82ef4e16d6fb3103
URL:
https://github.com/llvm/llvm-project/commit/51d0e40c2ee1ab25f408f22e82ef4e16d6fb3103
DIFF:
https://github.com/llvm/llvm-project/commit/51d0e40c2ee1ab25f408f22e82ef4e16d6fb3103.diff
L
arsenm wrote:
ping
https://github.com/llvm/llvm-project/pull/102462
___
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/111976
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/arsenm closed
https://github.com/llvm/llvm-project/pull/111975
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Matt Arsenault
Date: 2024-10-15T09:45:53+04:00
New Revision: 5b330a72a027fb557704a543010fa9656d547023
URL:
https://github.com/llvm/llvm-project/commit/5b330a72a027fb557704a543010fa9656d547023
DIFF:
https://github.com/llvm/llvm-project/commit/5b330a72a027fb557704a543010fa9656d547023.diff
awson wrote:
> If you have approvals you don't have to wait for other reviewers unless they
> have made a comment that required a change and you them to confirm the fix or
> they explicitly blocked the review.
I believe I have no write access to the repo.
Could you, please, merge this?
@cor3
thesamesam wrote:
cc @glaubitz (https://gcc.gnu.org/bugzilla/show_bug.cgi?id=113341)
https://github.com/llvm/llvm-project/pull/111995
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/awson updated https://github.com/llvm/llvm-project/pull/96464
>From 096b999120cc28844d780acbc16f8308b3a54160 Mon Sep 17 00:00:00 2001
From: awson
Date: Mon, 24 Jun 2024 10:34:51 +0300
Subject: [PATCH 1/3] [Clang][Sema] don't handle ArraySize/AllocType early.
---
clang/lib/Se
Author: Younan Zhang
Date: 2024-10-15T12:52:31+08:00
New Revision: 3733b0cff314e1427b1a905e6e7438b536f9a96d
URL:
https://github.com/llvm/llvm-project/commit/3733b0cff314e1427b1a905e6e7438b536f9a96d
DIFF:
https://github.com/llvm/llvm-project/commit/3733b0cff314e1427b1a905e6e7438b536f9a96d.diff
https://github.com/zyn0217 closed
https://github.com/llvm/llvm-project/pull/112177
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
arsenm wrote:
> I'm thinking we could have a language option like
> `-fimplicit-default-addrspace-cast` that allows all conversions to default.
> Then if we want users to be aware that this even exists we could suggest it
> in the error message if it would fix it.
>
> I do understand that the
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Thomas Fransham (fsfod)
Changes
Update clangDaemonTweaks cmake target to set explicit symbol visibility macros
to correct mode for windows.
This will fix linker duplicate symbols errors from clangDaemonTweaks exporting
clang symbols inste
https://github.com/fsfod created
https://github.com/llvm/llvm-project/pull/112304
Update clangDaemonTweaks cmake target to set explicit symbol visibility macros
to correct mode for windows.
This will fix linker duplicate symbols errors from clangDaemonTweaks exporting
clang symbols instead of
@@ -721,7 +721,9 @@ class Qualifiers {
// to implicitly cast into the default address space.
(A == LangAS::Default &&
(B == LangAS::cuda_constant || B == LangAS::cuda_device ||
- B == LangAS::cuda_shared));
+ B == LangAS
@@ -10057,15 +10057,18 @@ void ASTReader::finishPendingActions() {
// For a function defined inline within a class template, force the
// canonical definition to be the one inside the canonical definition of
// the template. This ensures that we instantiate fr
Author: Owen Pan
Date: 2024-10-14T19:29:44-07:00
New Revision: ee6468494e08651ec7f534781693408f34a74753
URL:
https://github.com/llvm/llvm-project/commit/ee6468494e08651ec7f534781693408f34a74753
DIFF:
https://github.com/llvm/llvm-project/commit/ee6468494e08651ec7f534781693408f34a74753.diff
LOG:
@@ -192,13 +201,13 @@ GetAffectingModuleMaps(const Preprocessor &PP, Module
*RootModule) {
// The containing module map is affecting, because it's being pointed
// into by Module::DefinitionLoc.
- if (auto FE = MM.getContainingModuleMapFile(Mod))
-Mod
Baiyi27 wrote:
> This feature works as intended, it is only for filtering out diagnosed
> issues, not for skipping the analysis in those files. See #52959 for the
> feature you are looking for. TLDR: clang-tidy currently does not skip the
> analysis in (system) header files, `exclude-header-fi
llvmbot wrote:
/pull-request llvm/llvm-project#112293
https://github.com/llvm/llvm-project/pull/111804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
zyn0217 wrote:
I'll go ahead and backport it - this bug was first introduced by sugar
preservation on CTAD guides included in the 19.x cycle.
/backport 0bc02b999a
https://github.com/llvm/llvm-project/pull/111804
___
cfe-commits mailing list
cfe-commi
https://github.com/zyn0217 milestoned
https://github.com/llvm/llvm-project/pull/111804
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/112177
>From 78657277495bbd1f82faef655c49320a4315272f Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Mon, 14 Oct 2024 17:42:17 +0800
Subject: [PATCH 1/4] [Clang] Fix a DeclContext mismatch when parsing nested
lambd
https://github.com/thevar1able updated
https://github.com/llvm/llvm-project/pull/112283
>From c9db5830f95727601900aa9a87c0970ecfbf5c76 Mon Sep 17 00:00:00 2001
From: Konstantin Bogdanov
Date: Tue, 15 Oct 2024 03:06:46 +0200
Subject: [PATCH] Fix a typo
---
.../CrtpConstructorAccessibilityCheck
https://github.com/ywgrit edited
https://github.com/llvm/llvm-project/pull/111488
___
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: Yihe Li (Mick235711)
Changes
Currently, the following program
```cpp
[[nodiscard]] int fun() { return 1; }
[[deprecated]] int fun2() { return 2; }
int main()
{
fun(); fun2();
}
```
generates the following diagnostics on Clang trunk: ([
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/Mick235711 created
https://github.com/llvm/llvm-project/pull/112289
Currently, the following program
```cpp
[[nodiscard]] int fun() { return 1; }
[[deprecated]] int fun2() { return 2; }
int main()
{
fun(); fun2();
}
```
generates the following diagnostics on Clang trunk:
llvmbot wrote:
@llvm/pr-subscribers-clang-codegen
@llvm/pr-subscribers-backend-aarch64
Author: Florian Mayer (fmayer)
Changes
This greatly simplifies the code, and makes sure no optimizations are
applied that assume the bigger alignment or size, which could be
incorrect if we link together
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Florian Mayer (fmayer)
Changes
This greatly simplifies the code, and makes sure no optimizations are
applied that assume the bigger alignment or size, which could be
incorrect if we link together with non-instrumented code.
---
Full diff:
https://github.com/fmayer ready_for_review
https://github.com/llvm/llvm-project/pull/111918
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
@@ -28,12 +28,24 @@
#include "clang/Analysis/FlowSensitive/MatchSwitch.h"
#include "clang/Analysis/FlowSensitive/TypeErasedDataflowAnalysis.h"
#include "clang/Analysis/FlowSensitive/WatchedLiteralsSolver.h"
+#include "clang/Support/Compiler.h"
#include "llvm/ADT/STLExtras.h"
https://github.com/ymand requested changes to this pull request.
https://github.com/llvm/llvm-project/pull/108051
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yus3710-fj wrote:
Gentle ping @MaskRay.
https://github.com/llvm/llvm-project/pull/110061
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/yabinc closed
https://github.com/llvm/llvm-project/pull/110051
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: yabinc
Date: 2024-10-14T16:32:24-07:00
New Revision: 627746581b8fde4143533937130f420bbbdf9ddf
URL:
https://github.com/llvm/llvm-project/commit/627746581b8fde4143533937130f420bbbdf9ddf
DIFF:
https://github.com/llvm/llvm-project/commit/627746581b8fde4143533937130f420bbbdf9ddf.diff
LOG: R
llvmbot wrote:
@llvm/pr-subscribers-clang-analysis
Author: Malavika Samak (malavikasamak)
Changes
Do not warn when a constant sized array is indexed with an expression that
contains bitwise and operation
involving constants and it always results in a bound safe access.
(rdar://136684050)
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Malavika Samak (malavikasamak)
Changes
Do not warn when a constant sized array is indexed with an expression that
contains bitwise and operation
involving constants and it always results in a bound safe access.
(rdar://136684050)
---
Ful
https://github.com/malavikasamak created
https://github.com/llvm/llvm-project/pull/112284
Do not warn when a constant sized array is indexed with an expression that
contains bitwise and operation
involving constants and it always results in a bound safe access.
(rdar://136684050)
>From b557d1
https://github.com/torshepherd updated
https://github.com/llvm/llvm-project/pull/95712
>From 2c9e7c16524b7ce3bf818e451279722dc45d3efc Mon Sep 17 00:00:00 2001
From: Tor Shepherd
Date: Mon, 23 Sep 2024 23:12:23 -0400
Subject: [PATCH 1/5] just defaults
---
clang-tools-extra/clangd/Config.h
torshepherd wrote:
Ok I found a couple of nontrivial issues that I took care of in the latest
commit.
1. Using `DefaultArguments` required having parameter names set to true as
well. I separated them so that both have a modality that works with or without
the other. See screenshot for default
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 39ac8b25cdca230509078d575d46c538bdf90e18
d336c456a93b36b1a15fd1cbf1a80275b2704525 --e
llvmbot wrote:
@llvm/pr-subscribers-clang-tools-extra
Author: Konstantin Bogdanov (thevar1able)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/112283.diff
2 Files Affected:
- (modified)
clang-tools-extra/clang-tidy/bugprone/CrtpConstructorAccessibilityCheck.cpp
(+1
llvmbot wrote:
@llvm/pr-subscribers-clang-tidy
Author: Konstantin Bogdanov (thevar1able)
Changes
---
Full diff: https://github.com/llvm/llvm-project/pull/112283.diff
2 Files Affected:
- (modified)
clang-tools-extra/clang-tidy/bugprone/CrtpConstructorAccessibilityCheck.cpp
(+1-1)
-
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/thevar1able created
https://github.com/llvm/llvm-project/pull/112283
None
>From 31e4d362b6466b101cdf896003c6c264786b3f2a Mon Sep 17 00:00:00 2001
From: Konstantin Bogdanov
Date: Tue, 15 Oct 2024 01:21:21 +0200
Subject: [PATCH] Fix a typo
---
.../CrtpConstructorAccessibilit
https://github.com/fsfod updated
https://github.com/llvm/llvm-project/pull/108051
>From e39aee57ce8a98723f40f67ae33808bc6213c61d Mon Sep 17 00:00:00 2001
From: Thomas Fransham
Date: Sat, 7 Sep 2024 15:53:09 +0100
Subject: [PATCH 1/4] [llvm] Support llvm::Any across shared libraries on
windows
@@ -0,0 +1,713 @@
+//===-- Mustache.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: Ap
@@ -0,0 +1,713 @@
+//===-- Mustache.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: Ap
@@ -0,0 +1,567 @@
+//===-- Mustache.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: Ap
@@ -0,0 +1,567 @@
+//===-- Mustache.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: Ap
jhuber6 wrote:
> So overall I'm leaning against doing it as is; guarding it under some flavour
> of `allow-relaxed-address-space-handling` or something that sounds better but
> is still noisy enough to make it clear there's danger seems preferable, at
> least to me.
A language option is reaso
nicovank wrote:
Rebase past 3484ed9325f30b56717a1b939af4c58dd07848e0 for green CI.
https://github.com/llvm/llvm-project/pull/112162
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Nicolas van Kempen
Date: 2024-10-14T18:43:33-04:00
New Revision: 3484ed9325f30b56717a1b939af4c58dd07848e0
URL:
https://github.com/llvm/llvm-project/commit/3484ed9325f30b56717a1b939af4c58dd07848e0
DIFF:
https://github.com/llvm/llvm-project/commit/3484ed9325f30b56717a1b939af4c58dd07848e0.
@@ -2,7 +2,7 @@
; RUN: llvm-as -o %t.o %s
; RUN: llvm-as -o %t2.o %S/Inputs/libcall-archive.ll
; RUN: llvm-ar rcs %t.a %t2.o
-; RUN: wasm-ld -o %t %t.o %t.a
+; RUN: wasm-ld -mllvm -mattr=-bulk-memory -o %t %t.o %t.a
sbc100 wrote:
Why is this needed? Should th
dpalermo wrote:
> Sorry, I opened a build fix PR here #112275
Thanks for the fix! Closing revert.
https://github.com/llvm/llvm-project/pull/112274
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo
https://github.com/dpalermo closed
https://github.com/llvm/llvm-project/pull/112274
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/a-tarasyuk updated
https://github.com/llvm/llvm-project/pull/112081
>From 67c41612085489a2a17eec49f98dbfa0e5bb97cf Mon Sep 17 00:00:00 2001
From: Oleksandr T
Date: Sat, 12 Oct 2024 08:27:51 +0300
Subject: [PATCH] [Clang] fix range calculation for conditionals with throw
expr
AlexVlx wrote:
> > Running into an observable situation where this is a concern means that
> > either you've messed around with (non C/C++) attributes, or are linking in
> > something exciting. Neither of which constitutes valid C/C++, and I'm
> > leaning towards saying shouldn't be allowed to
jhuber6 wrote:
> Running into an observable situation where this is a concern means that
> either you've messed around with (non C/C++) attributes, or are linking in
> something exciting. Neither of which constitutes valid C/C++, and I'm leaning
> towards saying shouldn't be allowed to silentl
AlexVlx wrote:
> I think the default AS is expected to be losslessly converted to other AS but
> not the other way around, though I understand it is not clearly stated in
> LangRef or other places and a lot of code just assumes it can.
I don't think this expectation is thoroughly encoded anywh
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 c4131cb56c9de616442525b1f02162eb470df1f8
f173e127c135f88ea37aef26010583078063c202 --e
kateinoigakukun wrote:
Sorry, I opened a build fix PR here
https://github.com/llvm/llvm-project/pull/112275
https://github.com/llvm/llvm-project/pull/112274
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman
https://github.com/ronlieb approved this pull request.
https://github.com/llvm/llvm-project/pull/112274
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
llvmbot wrote:
@llvm/pr-subscribers-mc
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library lib/liblldWasm.s
llvmbot wrote:
@llvm/pr-subscribers-pgo
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library lib/liblldWasm.
llvmbot wrote:
@llvm/pr-subscribers-lld
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library lib/liblldWasm.
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library lib/liblldWas
llvmbot wrote:
@llvm/pr-subscribers-backend-webassembly
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library
llvmbot wrote:
@llvm/pr-subscribers-lld-wasm
@llvm/pr-subscribers-clang-driver
Author: None (dpalermo)
Changes
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Lin
https://github.com/dpalermo created
https://github.com/llvm/llvm-project/pull/112274
Reverts llvm/llvm-project#111332
Unable to build clang/flang with OpenMP this patch. Results in:
https://lab.llvm.org/staging/#/builders/105/builds/6816
38.873 [112/2/7464] Linking CXX shared library lib/libl
https://github.com/sookach updated
https://github.com/llvm/llvm-project/pull/112111
>From d7a37cb7805d5ba982b8073c2485f16dec8a1eac Mon Sep 17 00:00:00 2001
From: Andrew Sukach
Date: Sat, 12 Oct 2024 19:47:30 -0400
Subject: [PATCH] [clang] Check for null TypeSourceInfo in
Sema::CreateUnaryExprO
@@ -5622,6 +5622,9 @@ void CodeGenModule::EmitGlobalVarDefinition(const VarDecl
*D,
getCUDARuntime().handleVarRegistration(D, *GV);
}
+ if (LangOpts.HLSL)
+getHLSLRuntime().handleGlobalVarDefinition(D, GV);
damyanp wrote:
Just checking you know th
@@ -489,3 +494,100 @@ void CGHLSLRuntime::generateGlobalCtorDtorCalls() {
GV->eraseFromParent();
}
}
+
+// Returns handle type of a resource, if the type is a resource
+// or an array of resources
+static const HLSLAttributedResourceType *findHandleTypeOnResource(QualTy
@@ -489,3 +494,100 @@ void CGHLSLRuntime::generateGlobalCtorDtorCalls() {
GV->eraseFromParent();
}
}
+
+// Returns handle type of a resource, if the type is a resource
+// or an array of resources
+static const HLSLAttributedResourceType *findHandleTypeOnResource(QualTy
@@ -1121,6 +1121,11 @@
CodeGenFunction::GenerateCXXGlobalInitFunc(llvm::Function *Fn,
if (Decls[i])
EmitRuntimeCall(Decls[i]);
+if (getLangOpts().HLSL)
+ if (llvm::Function *ResInitFn =
+ CGM.getHLSLRuntime().createResourceBindingInitFn())
1 - 100 of 579 matches
Mail list logo