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/2] [Clang][Sema] don't handle ArraySize/AllocType early.
---
clang/lib/Se
erichkeane wrote:
It seems reasonable to me, but I want @cor3ntin to review this, he's the one
who got his foot stuck in array bounds for a while :D
However, this patch needs a few changes:
1- A better patch title, perhaps improved commit message
2- Release note
https://github.com/llvm/llvm
https://github.com/AaronBallman commented:
Thank you for catching this! Can you please add a release note to
`clang/docs/ReleaseNotes.rst` so users know about the fix?
I think these changes otherwise look good.
https://github.com/llvm/llvm-project/pull/96464
___
awson wrote:
@erichkeane, @cor3ntin
I see you actively reviewed (related, failed) #89036.
Could you look into this?
The idea of the fix is extremely simple: if we want to do something once, but
mistakenly do it twice, just don't do it the first time.
https://github.com/llvm/llvm-project/pull
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/2] [Clang][Sema] don't handle ArraySize/AllocType early.
---
clang/lib/Se
https://github.com/awson edited 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
