jp4a50 added a comment.

In D146042#4195295 <https://reviews.llvm.org/D146042#4195295>, @owenpan wrote:

>> The previous implementation of the option involved a hack which corrupted 
>> the parenthesis state stack.
>
> Can you link the review (e.g. `Dnnnnnn`) of the previous implementation in 
> the summary?
>
>> Specifically, this change fixes github issues #55708, #53212, #52846 and 
>> #59954.
>
> Please link the issues (e.g. 
> https://github.com/llvm/llvm-project/issues/nnnnn).

Done! Please see updated description.

To give a little more relevant context, my team were the original authors of 
this feature/option (primarily for our own benefit) so we are motivated to fix 
it and get it right this time.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146042

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

Reply via email to