curdeius added a comment.
In D121450#3397861 <https://reviews.llvm.org/D121450#3397861>, @jgorbe wrote:
> Hi,
>
> We've observed that this patch introduces infinite loops in some cases.
> Here's a reduced test case:
>
> export class Foo extends Bar {
> get case(): Case {
> return (
> (this.Bar$has('case')) ? (this.Bar$get('case')) :
> (this.case = new Case()));
> }
> }
>
> Saving this as `/tmp/test.ts` and running `clang-format /tmp/test.ts` loops
> indefinitely with this patch (I stopped it after 1m27s) and finishes
> instantly (0.12s) after reverting the patch locally. I'm going to go ahead
> and push a revert. Please let me know if you need help reproducing the
> problem.
Thanks for the reproducer.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D121450/new/
https://reviews.llvm.org/D121450
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
