[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-14 Thread Tim Northover via cfe-commits
TNorthover wrote: Thanks. Good idea on the docs, I've added some wording and pushed the change (4299c727e480) https://github.com/llvm/llvm-project/pull/85054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailma

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-14 Thread Tim Northover via cfe-commits
https://github.com/TNorthover closed https://github.com/llvm/llvm-project/pull/85054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-13 Thread Daniel Kiss via cfe-commits
https://github.com/DanielKristofKiss approved this pull request. Maybe a line or two in the `clang/docs/LanguageExtensions.rst` would be useful. https://github.com/llvm/llvm-project/pull/85054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org htt

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-13 Thread Amara Emerson via cfe-commits
https://github.com/aemerson approved this pull request. LGTM. https://github.com/llvm/llvm-project/pull/85054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-13 Thread Jon Roelofs via cfe-commits
https://github.com/jroelofs approved this pull request. https://github.com/llvm/llvm-project/pull/85054 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-13 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang @llvm/pr-subscribers-clang-codegen Author: Tim Northover (TNorthover) Changes It's useful to provide an indicator code with the trap, which the generic __builtin_trap can't do. asm("brk #N") is an option, but following that with a __builtin_unrea

[clang] AArch64: add __builtin_arm_trap (PR #85054)

2024-03-13 Thread Tim Northover via cfe-commits
https://github.com/TNorthover created https://github.com/llvm/llvm-project/pull/85054 It's useful to provide an indicator code with the trap, which the generic __builtin_trap can't do. asm("brk #N") is an option, but following that with a __builtin_unreachable() leads to two traps when the com