[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-08-23 Thread Owen Pan via cfe-commits
owenca wrote: Bisected to 834ac2e205dd8e492d6084a7952e68e19a1f54db. @rymiel https://github.com/llvm/llvm-project/pull/96801 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-29 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `sanitizer-ppc64le-linux` running on `ppc64le-sanitizer` while building `clang` at step 2 "annotate". Full details are available at: https://lab.llvm.org/buildbot/#/builders/72/builds/596 Here is the relevant piece of the buil

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-29 Thread Emilia Kond via cfe-commits
https://github.com/rymiel closed https://github.com/llvm/llvm-project/pull/96801 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-28 Thread Owen Pan via cfe-commits
https://github.com/owenca approved this pull request. https://github.com/llvm/llvm-project/pull/96801 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-27 Thread Björn Schäpers via cfe-commits
https://github.com/HazardyKnusperkeks approved this pull request. https://github.com/llvm/llvm-project/pull/96801 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-27 Thread Björn Schäpers via cfe-commits
HazardyKnusperkeks wrote: > I'm not even sure if this change is a good idea! I think it's very likely it > will lead to regressions, for we don't have tests for such cases yet, it seems I'm sure you will look after them, so I'd say let them come. ;) https://github.com/llvm/llvm-project/pull/96

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-26 Thread Emilia Kond via cfe-commits
rymiel wrote: I'm not even sure if this change is a good idea! I think it's very likely it will lead to regressions, for we don't have tests for such cases yet, it seems https://github.com/llvm/llvm-project/pull/96801 ___ cfe-commits mailing list cfe-

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-26 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang-format Author: Emilia Kond (rymiel) Changes Currently, question mark and colon tokens are not allowed between angle brackets, as a template argument, if we are in an expression context. However, expressions can still allowed in non-expression con

[clang] [clang-format] Allow ternary in all templates (PR #96801)

2024-06-26 Thread Emilia Kond via cfe-commits
https://github.com/rymiel created https://github.com/llvm/llvm-project/pull/96801 Currently, question mark and colon tokens are not allowed between angle brackets, as a template argument, if we are in an expression context. However, expressions can still allowed in non-expression contexts, lea