@@ -0,0 +1,425 @@
+==
+DXIL Resource Handling
+==
+
+.. contents::
+ :local:
+
+.. toctree::
+ :hidden:
+
+Introduction
+
+
+Resources in DXIL are represented via ``TargetExtType`` in LLVM IR and
+eventually lowered by the Di
https://github.com/momchil-velikov created
https://github.com/llvm/llvm-project/pull/99063
This patch implements these intrinsics:
``` c
// Variants are also available for:
// [_f32_x2], [_f64_x2],
// [_f16_x4], [_f32_x4], [_f64_x4]
svfloat16x2_t svamax[_f16_x2](svfloat16x2 zd, svfloa
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Momchil Velikov (momchil-velikov)
Changes
This patch implements these intrinsics:
``` c
// Variants are also available for:
// [_f32_x2], [_f64_x2],
// [_f16_x4], [_f32_x4], [_f64_x4]
svfloat16x2_t svamax[_f16_x2](svfloat16x2 zd,
llvmbot wrote:
@llvm/pr-subscribers-backend-aarch64
Author: Momchil Velikov (momchil-velikov)
Changes
This patch implements these intrinsics:
``` c
// Variants are also available for:
// [_f32_x2], [_f64_x2],
// [_f16_x4], [_f32_x4], [_f64_x4]
svfloat16x2_t svamax[_f16_x2](svflo
@@ -670,6 +671,7 @@ llvm::Error ClangDocBitcodeReader::readRecord(unsigned ID,
T I) {
template <>
llvm::Error ClangDocBitcodeReader::readRecord(unsigned ID, Reference *I) {
+ llvm::TimeTraceScope("Reducing infos", "readRecord");
ilovepi wrote:
I know this i
https://github.com/ilovepi edited
https://github.com/llvm/llvm-project/pull/97644
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/ilovepi approved this pull request.
LGTM modulo a small nit in the naming, we can always add more instrumentation
if we need it, but this is a good start.
https://github.com/llvm/llvm-project/pull/97644
___
cfe-commits mailing list
ilovepi wrote:
> When sorted by USR, there are no differences between the index between this
> patch and what's on main,
I'm not sure I completely follow. are you saying that when you sort the
contents of all the files you no longer have any differences? So you're sure
the only differences i
https://github.com/efriedma-quic commented:
What's the relationship between the max vector alignment, and choosing whether
to return a vector directly? As long as clang isn't emitting the byval
attribute, I don't think it's the frontend's problem.
https://github.com/llvm/llvm-project/pull/986
@@ -0,0 +1,8 @@
+// RUN: not %clang_cc1 -triple riscv64 -target-feature +i -emit-llvm -o - %s
2>&1 | FileCheck %s --check-prefix=CHECK-UNSUPPORT-OS
topperc wrote:
"invalid" in test name is misspelled
https://github.com/llvm/llvm-project/pull/99040
_
@@ -324,6 +324,9 @@ ARMTargetInfo::ARMTargetInfo(const llvm::Triple &Triple,
case llvm::Triple::GNU:
setABI("apcs-gnu");
break;
+case llvm::Triple::PAuthTest:
asl wrote:
Do we really need to touch 32-bit ARM?
https://github.com/llvm/llvm-p
https://github.com/tahonermann commented:
As far as I can tell, `ObjCMethodList` is already self-assignment safe so long
as `PointerIntPair` is. It looks like the latter might not be though since its
copy assignment operator (as implemented via `PunnedPointer`) uses `memcpy` and
it does not gu
https://github.com/shafik commented:
This LGTM but when someone blocks you should wait for approval before landing
the change.
https://github.com/llvm/llvm-project/pull/98699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.o
https://github.com/shafik edited https://github.com/llvm/llvm-project/pull/98699
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/banach-space approved this pull request.
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tahonermann approved this pull request.
Looks good! Thanks @smanna12!
https://github.com/llvm/llvm-project/pull/97912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
smanna12 wrote:
Thank you @tahonermann for reviews!
https://github.com/llvm/llvm-project/pull/97912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: smanna12
Date: 2024-07-16T12:24:57-05:00
New Revision: ca61bdde476aae4d179f16561ce4d245df9f0fdf
URL:
https://github.com/llvm/llvm-project/commit/ca61bdde476aae4d179f16561ce4d245df9f0fdf
DIFF:
https://github.com/llvm/llvm-project/commit/ca61bdde476aae4d179f16561ce4d245df9f0fdf.diff
LOG:
https://github.com/smanna12 closed
https://github.com/llvm/llvm-project/pull/97912
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/efriedma-quic approved this pull request.
LGTM
Please don't use `[test]` as a prefix in commit messages; just use the same
prefix you'd use for the code change, e.g. `[clang codegen]`
https://github.com/llvm/llvm-project/pull/98704
___
rjmccall wrote:
I agree that blocks don't belong in the ObjC segment.
https://github.com/llvm/llvm-project/pull/97172
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/smanna12 updated
https://github.com/llvm/llvm-project/pull/97902
>From 1f80c0a172b58ad15d6b1dce02b63ac682bc7dc0 Mon Sep 17 00:00:00 2001
From: "Manna, Soumi"
Date: Sat, 6 Jul 2024 09:03:15 -0700
Subject: [PATCH 1/5] [Clang] Remove unnecessary copy
Reported by Static Analyzer
@@ -1551,7 +1551,7 @@ ASTNodeImporter::VisitCountAttributedType(const
CountAttributedType *T) {
Expr *CountExpr = importChecked(Err, T->getCountExpr());
SmallVector CoupledDecls;
- for (auto TI : T->dependent_decls()) {
+ for (const TypeCoupledDeclRefInfo &TI : T->depen
https://github.com/smanna12 edited
https://github.com/llvm/llvm-project/pull/97902
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
yuxuanchen1997 wrote:
@ChuanqiXu9
> I feel the name `[[coro_inplace_task]]` is not so good. Since it doesn't
> happen unconditionally. It happens conditionally. I still want to try to make
> it clear in the name. I still like names like `[[coro_elide_after_await]]`,
> which I feel is more cl
smanna12 wrote:
> As far as I can tell, `ObjCMethodList` is already self-assignment safe so
> long as `PointerIntPair` is. It looks like the latter might not be though
> since its copy assignment operator (as implemented via `PunnedPointer`) uses
> `memcpy` and it does not guard against self-a
https://github.com/al45tair updated
https://github.com/llvm/llvm-project/pull/97597
>From c11624370bf5615c7902993052435b92ae50b41b Mon Sep 17 00:00:00 2001
From: Alastair Houghton
Date: Wed, 3 Jul 2024 17:00:51 +0100
Subject: [PATCH 1/7] [Parser][ObjC] Add -Wobjc-prefix-length warning option.
https://github.com/tahonermann approved this pull request.
Thanks @smanna12. I think this looks ok; returning null here does appear to be
consistent with other overloads of `CreateTypeDefinition` for `RecordType` and
`ObjCInterfaceType`. I agree with @Michael137 that it would be nice to have an
AaronBallman wrote:
> Could you add a flang driver test as well?
The RFC was specifically about `-Ofast` in Clang and not behavior in Flang, so
what test behavior would you like to see? This patch currently deprecates for
both Clang and Flang, is that acceptable?
https://github.com/llvm/llvm-
https://github.com/PiotrZSL approved this pull request.
LGTM, some tests for rotate_copy and implace_merge would be welcome but it
isn't must as it work similar to rotate.
https://github.com/llvm/llvm-project/pull/99057
___
cfe-commits mailing list
cf
https://github.com/yuxuanchen1997 updated
https://github.com/llvm/llvm-project/pull/98971
>From 628f30e84d8b5a708f3a88576d18f290429e4c88 Mon Sep 17 00:00:00 2001
From: Yuxuan Chen
Date: Tue, 4 Jun 2024 23:22:00 -0700
Subject: [PATCH] [Clang] Introduce [[clang::coro_inplace_task]]
Implement noa
DavidTruby wrote:
I don't know why the pre-commit build failed on Windows. The build itself looks
fine but seems to be complaining about slow tests?
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.or
https://github.com/topperc created
https://github.com/llvm/llvm-project/pull/99070
This matches their documentation and the capitalization we use for the
RISCVSubtarget methods.
>From 40ee0ece0118bd074919b88fc404446e6db4df46 Mon Sep 17 00:00:00 2001
From: Craig Topper
Date: Tue, 16 Jul 2024 1
llvmbot wrote:
@llvm/pr-subscribers-backend-risc-v
@llvm/pr-subscribers-clang
Author: Craig Topper (topperc)
Changes
This matches their documentation and the capitalization we use for the
RISCVSubtarget methods.
---
Full diff: https://github.com/llvm/llvm-project/pull/99070.diff
3 File
https://github.com/tahonermann edited
https://github.com/llvm/llvm-project/pull/94368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/tahonermann commented:
Thanks @smanna12. I'm uncertain about this change; see my additional comment.
https://github.com/llvm/llvm-project/pull/94368
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
@@ -13681,12 +13681,13 @@ void Sema::AddInitializerToDecl(Decl *RealDecl, Expr
*Init, bool DirectInit) {
}
Init = Result.getAs();
+assert(Init && "Init must not be null");
+
IsParenListInit = !InitSeq.steps().empty() &&
InitSeq.step_beg
https://github.com/llvm-beanz approved this pull request.
https://github.com/llvm/llvm-project/pull/90553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
steven-johnson wrote:
After some initial experiments, it looks unlikely that a quick fix is possible
on our side. May I suggest a revert with a (say) 30-day expiration date? That
should give us enough time to come up with a fix on our side (which will
actually be a drop-other-work-and-fix-it a
thurstond wrote:
Updated to IR tests
https://github.com/llvm/llvm-project/pull/98247
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ceseo wrote:
> I don't know why the pre-commit build failed on Windows. The build itself
> looks fine but seems to be complaining about slow tests?
Actually, the error is here:
```
FAIL: Flang :: Driver/linker-flags.f90 (218 of 2807)
TEST 'Flang :: Driver/linker-flags.f90'
https://github.com/zahiraam updated
https://github.com/llvm/llvm-project/pull/98520
>From da863999000a6b12f2930247de9df3daf0e5a608 Mon Sep 17 00:00:00 2001
From: Zahira Ammarguellat
Date: Thu, 11 Jul 2024 11:54:13 -0700
Subject: [PATCH 1/7] The pragma STDC CX_LIMITED_RANGE ON should have
prece
https://github.com/ceseo commented:
linker-flags test seems to be failing on Windows.
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/preames approved this pull request.
LGTM
https://github.com/llvm/llvm-project/pull/99070
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/smanna12 updated
https://github.com/llvm/llvm-project/pull/97220
>From 63f45c952ff8ab7df261a150355a34267e4a645c Mon Sep 17 00:00:00 2001
From: "Manna, Soumi"
Date: Sun, 30 Jun 2024 08:40:27 -0700
Subject: [PATCH 1/4] [Clang] Prevent null pointer dereference in designated
ini
https://github.com/yichi170 created
https://github.com/llvm/llvm-project/pull/99075
In `clang/lib/Lex/PPMacroExpansion.cpp`, replaced the usage of the obsolete
`asctime` function with `strftime` for generating timestamp strings.
This is my first time contributing to an open-source project. If
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
Author: Yi-Chi Lee (yichi170)
Changes
In `clang/lib/Lex/PPMacroExpansion.cpp`, replaced the usage of the obsolete
`asctime` function with `strftime` for generating timestamp strings.
This is my first time contributing to an open-source project. I
@@ -3206,9 +3206,11 @@ ExprResult SemaObjC::BuildInstanceMessage(
}
if (!isDesignatedInitChain) {
const ObjCMethodDecl *InitMethod = nullptr;
+ auto *CurMD = SemaRef.getCurMethodDecl();
+ if (!CurMD)
+return ExprResult((Expr *)nullptr);
-
https://github.com/premanandrao updated
https://github.com/llvm/llvm-project/pull/97619
>From e1076942f07b1d2de857688c88de6ff966861650 Mon Sep 17 00:00:00 2001
From: Premanand M Rao
Date: Wed, 3 Jul 2024 11:20:42 -0700
Subject: [PATCH 1/2] [clang] Diagnose use of deprecated template alias
Issu
https://github.com/zahiraam updated
https://github.com/llvm/llvm-project/pull/98520
>From da863999000a6b12f2930247de9df3daf0e5a608 Mon Sep 17 00:00:00 2001
From: Zahira Ammarguellat
Date: Thu, 11 Jul 2024 11:54:13 -0700
Subject: [PATCH 1/8] The pragma STDC CX_LIMITED_RANGE ON should have
prece
@@ -0,0 +1,58 @@
+// RUN: %clang_cc1 -std=c++11 -fsyntax-only -verify %s
+//
+// This test checks that a deprecated attribute on an alias
+// template triggers a warning diagnostic when it is used.
+
+template
+struct NoAttr {
+ void foo() {}
+};
+
+// expected-note@+2 5{{'Using
https://github.com/smanna12 edited
https://github.com/llvm/llvm-project/pull/97220
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/PiotrZSL updated
https://github.com/llvm/llvm-project/pull/94869
>From ceeb4006d54b40a226a7a1f4f78f7f5f9d9dd7a1 Mon Sep 17 00:00:00 2001
From: Piotr Zegar
Date: Tue, 16 Jul 2024 18:34:25 +
Subject: [PATCH] [clang-tidy] Fix smart pointers handling in
bugprone-use-after-mo
https://github.com/PiotrZSL edited
https://github.com/llvm/llvm-project/pull/94869
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
kiranchandramohan wrote:
> > Could you add a flang driver test as well?
>
> The RFC was specifically about `-Ofast` in Clang and not behavior in Flang,
> so what test behavior would you like to see? This patch currently deprecates
> for both Clang and Flang, is that acceptable?
I was asking f
AaronBallman wrote:
> > > Could you add a flang driver test as well?
> >
> >
> > The RFC was specifically about `-Ofast` in Clang and not behavior in Flang,
> > so what test behavior would you like to see? This patch currently
> > deprecates for both Clang and Flang, is that acceptable?
>
>
AaronBallman wrote:
> After some initial experiments, it looks unlikely that a quick fix is
> possible on our side. May I suggest a revert with a (say) 30-day expiration
> date? That should give us enough time to come up with a fix on our side
> (which will actually be a drop-other-work-and-fi
Author: Aaron Ballman
Date: 2024-07-16T14:47:09-04:00
New Revision: d3f8105c65046173e20c4c59394b4a7f1bbe7627
URL:
https://github.com/llvm/llvm-project/commit/d3f8105c65046173e20c4c59394b4a7f1bbe7627
DIFF:
https://github.com/llvm/llvm-project/commit/d3f8105c65046173e20c4c59394b4a7f1bbe7627.diff
https://github.com/AaronBallman closed
https://github.com/llvm/llvm-project/pull/99079
___
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-codegen
@llvm/pr-subscribers-clang
Author: Aaron Ballman (AaronBallman)
Changes
Reverts llvm/llvm-project#98497
We're reverting this for approx 30 days so that the Halide project has time to
transition off the target.
---
Full diff: https://gith
AaronBallman wrote:
d3f8105c65046173e20c4c59394b4a7f1bbe7627 restores the targets, thank you all
for the discussion!
https://github.com/llvm/llvm-project/pull/98497
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin
efriedma-quic wrote:
> I was under the impression those bits were still needed for Chromium. CC
> @dschuff
Oh, clang/lib/CodeGen/Targets/PNaCl.cpp is still live for mipsel-nacl target
triple. Which we also have zero tests for, but I guess that's a separate issue.
https://github.com/llvm/llvm
dtcxzyw wrote:
I don't see any performance benefits based on IR diff or instruction count.
Changes on code size are mainly affected by JumpThreading/Inlining.
But I don't object to propagating more attributes if the compile-time impact is
negligible :)
https://github.com/llvm/llvm-project/p
njames93 wrote:
> LGTM, some tests for rotate_copy and implace_merge would be welcome but it
> isn't must as it work similar to rotate.
Yeah, like with the other supported algorithms, I didn't see the need to add a
test for every single function, as long as there's a test for each kind of
fun
https://github.com/mizvekov approved this pull request.
LGTM as well
https://github.com/llvm/llvm-project/pull/98671
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
vortex73 wrote:
@haberman Please review.
https://github.com/llvm/llvm-project/pull/98965
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
HazardyKnusperkeks wrote:
> It seems that this patch implements the new option without regard to the
> relative precedence of the binary operators:
>
> ```
> $ cat bar.cc
> f1 = f2 * f3 +
> f4 / f5 - f6;
>
https://github.com/sharadhr updated
https://github.com/llvm/llvm-project/pull/98761
>From 1fed92a00f0d732a2575861c2bf6a6d053407255 Mon Sep 17 00:00:00 2001
From: Sharadh Rajaraman
Date: Sat, 13 Jul 2024 19:25:47 +0100
Subject: [PATCH 1/4] Allow `--precompile` and `-fprebuilt-module-path` to be
https://github.com/matthew-f created
https://github.com/llvm/llvm-project/pull/99084
The regular expressions match functions that aren't anchored in the global
namespace. For example `::connect` matches `QObject::connect` This change is to
remove these false positives
>From d5aa95877b2c33189b
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-tools-extra
Author: None (matthew-f)
Changes
The regular expressions match functions that aren't anchored in the global
namespace. For example `::connect` matches `QObject::connect` This change is to
remove these false positives
---
Full diff: h
@@ -333,7 +338,8 @@ void CodeGenFunction::registerGlobalDtorWithLLVM(const
VarDecl &VD,
llvm::FunctionCallee Dtor,
llvm::Constant *Addr) {
// Create a function which calls the d
https://github.com/asl approved this pull request.
https://github.com/llvm/llvm-project/pull/98276
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
https://github.com/kovdan01 updated
https://github.com/llvm/llvm-project/pull/97237
>From 3b4b1b1739b810d758e68f30c48b648963cff740 Mon Sep 17 00:00:00 2001
From: Daniil Kovalev
Date: Mon, 1 Jul 2024 00:50:21 +0300
Subject: [PATCH 1/7] [PAC][Driver] Implement `-mbranch-protection=pauthabi`
opti
@@ -324,6 +324,9 @@ ARMTargetInfo::ARMTargetInfo(const llvm::Triple &Triple,
case llvm::Triple::GNU:
setABI("apcs-gnu");
break;
+case llvm::Triple::PAuthTest:
kovdan01 wrote:
No, we do not need that, thanks for bringing attention to this.
kkwli wrote:
The linker-flags test fails on AIX too.
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
AaronBallman wrote:
> The part I'm confused about is that this commit doesn't appear to simply be
> adding "or `alignas`" to some code that said "if `_Alignas`". Yes, the newly
> added `isAlignas()` checks do cover both attributes, but `_Alignas` was still
> working even after
> [b9cf7f1](htt
alexey-bataev wrote:
> In clang/lib/Sema/SemaOpenMP.cpp I have retained the marking of the inner
> regions of the associated statements with AssumeAttr. This will make the work
> in Codegen easier.
How it will make the codegen easier? It is better to try to avoid post-building
attribute assig
https://github.com/Endilll updated
https://github.com/llvm/llvm-project/pull/98736
>From 44b4a682f2b3d7e140f7b1bd124e7aabdbf439ad Mon Sep 17 00:00:00 2001
From: Vlad Serebrennikov
Date: Sat, 13 Jul 2024 13:10:25 +0300
Subject: [PATCH 1/5] [clang] Add deprecation warning for `-Ofast` driver
opt
https://github.com/DavidTruby updated
https://github.com/llvm/llvm-project/pull/99058
>From 66e13f92a0680742b552fabde25df7752f8510c9 Mon Sep 17 00:00:00 2001
From: David Truby
Date: Tue, 16 Jul 2024 15:39:28 +
Subject: [PATCH 1/2] [flang] Add -rtlib flag
This patch allows the -rtlib flag w
DavidTruby wrote:
It looks like clang_rt.crtbegin and clang_rt.crtend aren't always used on every
platform so I've just removed the check for those. The check for
`libclang_rt.builtins` and NOT `libgcc` and `libgcc_s` should be enough to
check we are linking the right thing.
https://github.co
https://github.com/bazuzi created
https://github.com/llvm/llvm-project/pull/99236
rewritten.
>From 7c75181c590f30cc439af6fa523f3ca78f8a933d Mon Sep 17 00:00:00 2001
From: Samira Bazuzi
Date: Tue, 16 Jul 2024 16:25:32 -0400
Subject: [PATCH] [clang][dataflow] Only propagate past CXXDefaultInitEx
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Samira Bazuzi (bazuzi)
Changes
rewritten.
---
Full diff: https://github.com/llvm/llvm-project/pull/99236.diff
2 Files Affected:
- (modified) clang/lib/Analysis/FlowSensitive/DataflowEnvironment.cpp (+6-1)
- (modified) clang/unittests/A
Author: Justin Bogner
Date: 2024-07-16T13:42:55-07:00
New Revision: 9be5f4f5d5617d11e96fe53e8c3f463252486641
URL:
https://github.com/llvm/llvm-project/commit/9be5f4f5d5617d11e96fe53e8c3f463252486641
DIFF:
https://github.com/llvm/llvm-project/commit/9be5f4f5d5617d11e96fe53e8c3f463252486641.diff
https://github.com/bogner closed https://github.com/llvm/llvm-project/pull/90553
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
luporl wrote:
In my debug build, clang is receiving a SIGSEGV in frame 7 below (that is
actually frame 0, as the frames below it are from a gdb call that received a
signal). In it, `this` holds an invalid address.
In frame 8, `NE` is 0x3fff, which probably caused the invalid access.
In fram
pskrgag wrote:
Hello @steakhal,
could you, please, take a look? I just found linked issue quite interesting and
decided to give it a try. I am new to llvm world, so I would really appreciate
your review
https://github.com/llvm/llvm-project/pull/98941
_
ojhunt wrote:
I notice the r+, but I haven't fully addressed this feedback - are you
suggesting corrections in a follow up patch, or just conceptually approving
with this PR? :D
https://github.com/llvm/llvm-project/pull/98276
___
cfe-commits mailing
@@ -465,7 +465,12 @@ class ResultObjectVisitor : public
AnalysisASTVisitor {
}
if (auto *DIE = dyn_cast(E)) {
- PropagateResultObject(DIE->getExpr(), Loc);
+ // If it has a rewritten init, we should propagate to that. If it
doesn't,
+ // then the CXXDe
https://github.com/yuxuanchen1997 updated
https://github.com/llvm/llvm-project/pull/98971
>From d38f0608753417c6b79e0684cb9dc23933dc957b Mon Sep 17 00:00:00 2001
From: Yuxuan Chen
Date: Tue, 4 Jun 2024 23:22:00 -0700
Subject: [PATCH] [Clang] Introduce [[clang::coro_inplace_task]]
Implement noa
gamesh411 wrote:
The warning just ''Type ProgramStatePartialTrait is incomplete", and to
reproduce it here is minimal exmaple:
``
https://github.com/llvm/llvm-project/pull/98150
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://list
https://github.com/ceseo approved this pull request.
Yes, this should be enough.
https://github.com/llvm/llvm-project/pull/99058
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
asl wrote:
> I notice the r+, but I haven't fully addressed this feedback - are you
> suggesting corrections in a follow up patch, or just conceptually approving
> with this PR? :D
I think this patch is fine to go as-is. We can re-consider that `MaybeSigned`
abstraction further on if it could
https://github.com/petrhosek created
https://github.com/llvm/llvm-project/pull/99246
Using single quotes made the build to break on Windows.
>From 526401e3da96c366f04ba98cfb453a72fd67cc76 Mon Sep 17 00:00:00 2001
From: Petr Hosek
Date: Tue, 16 Jul 2024 14:47:58 -0700
Subject: [PATCH] [CMake][F
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Petr Hosek (petrhosek)
Changes
Using single quotes made the build to break on Windows.
---
Full diff: https://github.com/llvm/llvm-project/pull/99246.diff
1 Files Affected:
- (modified) clang/cmake/caches/Fuchsia-stage2.cmake (+2-2)
@@ -459,7 +467,475 @@ void SemaHLSL::handleResourceClassAttr(Decl *D, const
ParsedAttr &AL) {
D->addAttr(HLSLResourceClassAttr::Create(getASTContext(), RC, ArgLoc));
}
-void SemaHLSL::handleResourceBindingAttr(Decl *D, const ParsedAttr &AL) {
+struct RegisterBindingFlags {
https://github.com/zeroomega approved this pull request.
https://github.com/llvm/llvm-project/pull/99246
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Author: Petr Hosek
Date: 2024-07-16T14:58:33-07:00
New Revision: f761e50a98f4123f6a57d8e9c3be64e5e39a16fe
URL:
https://github.com/llvm/llvm-project/commit/f761e50a98f4123f6a57d8e9c3be64e5e39a16fe
DIFF:
https://github.com/llvm/llvm-project/commit/f761e50a98f4123f6a57d8e9c3be64e5e39a16fe.diff
LO
https://github.com/zeroomega closed
https://github.com/llvm/llvm-project/pull/99246
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
201 - 300 of 443 matches
Mail list logo
