fhahn added a comment.

In D122768#3700486 <https://reviews.llvm.org/D122768#3700486>, @cor3ntin wrote:

> In D122768#3700451 <https://reviews.llvm.org/D122768#3700451>, @aeubanks 
> wrote:
>
>> in the commit message
>>
>>   In addition, capturing an anonymous union member,
>>   a bitfield, or a structured binding thereof now has a
>>   better diagnostic.
>>
>> "better diagnostic" implies there was already a diagnostic, but we're seeing 
>> new errors pop up. is this just a not precise commit message or are new 
>> errors related to capturing bitfields by reference unintentional?
>
> New errors when capturing bit fields are probably*not* intentional. You have 
> some of the problematic code i could look at? (not that the patch was 
> reverted and then re-landed to fix a bug there)
> And... the commit message is a bit inaccurate as the change that was 
> supposedly improving diagnostics for reference to bit fields was remove (and 
> i forgot to update the commit message accordingly)

It looks like this may cause https://github.com/llvm/llvm-project/issues/56909. 
It would eb great if you could take a look


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D122768/new/

https://reviews.llvm.org/D122768

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to