[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-04-05 Thread Erich Keane via cfe-commits
https://github.com/erichkeane approved this pull request. This seems reasonable to me. All of the changes look sound. Good job finding room here! https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailing list cfe-commits@lists.llvm.or

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread Shafik Yaghmour via cfe-commits
https://github.com/shafik commented: I think you need to update this too, unless I missed something: https://clang.llvm.org/docs/UsersManual.html#controlling-implementation-limits https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailin

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread Erich Keane via cfe-commits
https://github.com/erichkeane edited https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread via cfe-commits
cor3ntin wrote: Thanks @erichkeane. Let see if the bots scream https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread via cfe-commits
https://github.com/cor3ntin closed https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread Erich Keane via cfe-commits
https://github.com/erichkeane commented: This seems reasonable to me. All of the changes look sound. Good job finding room here! https://github.com/llvm/llvm-project/pull/132021 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.l

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread Erich Keane via cfe-commits
@@ -169,6 +169,8 @@ Modified Compiler Flags the behavior of ``-mtp`` in gcc. This changes the default behavior for ARM targets that provide the ``TPIDRURO`` register as this will be used instead of a call to the ``__aeabi_read_tp``. Programs that use ``__aeabi_read_tp`` bu

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: cor3ntin (cor3ntin) Changes This iterates on #104717 (which we had to revert) In a bid to increase our chances of success, we try to avoid blowing up the stack by - Using `runWithSufficientStackSpace` in ParseCompoundStatement - Reduci

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread via cfe-commits
cor3ntin wrote: Note this doesn't appear to impact performance negatively https://llvm-compile-time-tracker.com/compare.php?from=33e5d013b7f7a6ae136a058f842b30c87623ecfb&to=9e96b0f696f2478cdfeccec0c349b973e96f1432&stat=instructions%3Au https://github.com/llvm/llvm-project/pull/132021 __

[clang] [Clang] Increase the default expression nesting limit (PR #132021)

2025-03-19 Thread via cfe-commits
https://github.com/cor3ntin created https://github.com/llvm/llvm-project/pull/132021 This iterates on #104717 (which we had to revert) In a bid to increase our chances of success, we try to avoid blowing up the stack by - Using `runWithSufficientStackSpace` in ParseCompoundStatement - Reduc