FreddyLeaf wrote:
ping for review
https://github.com/llvm/llvm-project/pull/79086
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Endilll wrote:
> For us, these crashes block a solid piece of internal testing, thus new
> unnoticed errors accumulate and make things exponentially harder for the
> future.
If you have a solid piece of internal testing blocked by this, can you provide
us with more data? Ideally a working rep
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/80023
>From 58ceefe09cd992c3692bb3af7c2807ac8949ba67 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Thu, 1 Feb 2024 09:11:27 +0100
Subject: [PATCH
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/80023
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
https://github.com/tbaederr edited
https://github.com/llvm/llvm-project/pull/80023
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
tbaederr wrote:
I will also merge https://github.com/llvm/llvm-project/pull/80023 once the CI
is green. That will fix one potential source of problems.
https://github.com/llvm/llvm-project/pull/66514
___
cfe-commits mailing list
cfe-commits@lists.llvm
https://github.com/Pierre-vh updated
https://github.com/llvm/llvm-project/pull/76954
>From a967fdae9a8557331d2a228f391f39f9e27e8943 Mon Sep 17 00:00:00 2001
From: pvanhout
Date: Thu, 4 Jan 2024 14:12:00 +0100
Subject: [PATCH 1/3] [AMDGPU] Introduce Code Object V6
Introduce Code Object V6 in Cl
https://github.com/zyn0217 updated
https://github.com/llvm/llvm-project/pull/80395
>From 24a8251aab9a1a03953412ae47508d039d926212 Mon Sep 17 00:00:00 2001
From: Younan Zhang
Date: Fri, 2 Feb 2024 15:24:13 +0800
Subject: [PATCH 1/2] [clang][Sema] Fix GH58548
---
clang/docs/ReleaseNotes.rst
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -109,6 +109,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
bool VisitCXXScalarValueInitExpr(const CXXScalarValueInitExpr *E);
bool VisitSizeOfPackExpr(const SizeOfPackExpr *E);
bool VisitGenericSelection
Timm =?utf-8?q?Bäder?=
Message-ID:
In-Reply-To:
@@ -109,6 +109,7 @@ class ByteCodeExprGen : public
ConstStmtVisitor, bool>,
bool VisitCXXScalarValueInitExpr(const CXXScalarValueInitExpr *E);
bool VisitSizeOfPackExpr(const SizeOfPackExpr *E);
bool VisitGenericSelection
Author: Timm Bäder
Date: 2024-02-02T08:20:52+01:00
New Revision: 58ceefe09cd992c3692bb3af7c2807ac8949ba67
URL:
https://github.com/llvm/llvm-project/commit/58ceefe09cd992c3692bb3af7c2807ac8949ba67
DIFF:
https://github.com/llvm/llvm-project/commit/58ceefe09cd992c3692bb3af7c2807ac8949ba67.diff
LO
https://github.com/tbaederr updated
https://github.com/llvm/llvm-project/pull/80023
>From 046ac37551071c226ce155d25241d6676133d208 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Timm=20B=C3=A4der?=
Date: Tue, 30 Jan 2024 17:04:23 +0100
Subject: [PATCH] [clang] Fix a possible out-of-bounds read
Fixes
joker-eph wrote:
> I think you are assuming guarantees here that doesn't exist.
I'm just going by you're writing :)
You wrote: "To be able to flag incompatible bytecode files rather than have it
fail later in mysterious ways"
But my take is that you're not actually providing the ability to fla
HighCommander4 wrote:
Ok, I see. (I was confused because nothing in the patch looks at the contents
of `Symbol::DocComment` other than an `empty()` check; maybe a `bool
HasDocComment` flag is sufficient?)
I'll have a more detailed look when I get a chance, but one suggestion I wanted
to make
eaeltsin wrote:
> If you have a solid piece of internal testing blocked by this, can you
> provide us with more data? Ideally a working reproducer. This would be more
> productive than just reverting.
Sure, we are working on this and will share if we are successful. But please
take into accou
https://github.com/Endilll approved this pull request.
https://github.com/llvm/llvm-project/pull/80023
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
ckandeler wrote:
> Ok, I see. (I was confused because nothing in the patch looks at the contents
> of `Symbol::DocComment` other than
> an `empty()` check; maybe a `bool HasDocComment` flag is sufficient?)
Right, we just need to save the information whether there was a doc comment
before clan
501 - 517 of 517 matches
Mail list logo