[PATCH] D134365: [LoongArch] Add support for llvm.trap and llvm.debugtrap

Fri Sep 23 03:14:13 PDT 2022

xry111 added a comment.

In D134365#3811102 <https://reviews.llvm.org/D134365#3811102>, @xen0n wrote:

>> But again I'm not sure if we want a SIGTRAP of SIGILL (or something else) here.
>
> I'd really like to differentiate but according to glibc the `break 0` is used as the ultimate fallback for `abort`. Still a shame the two things have to be conflated if strictly implementing the specs...

And GCC is generating `break 0` for `__builtin_trap()`, while LLVM translates it to `llvm.trap` (not `debugtrap`).

Perhaps using `break 0` for both is OK, but I just wonder if there is a better way.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134365