diff options
author | Shuxin Yang <shuxin.llvm@gmail.com> | 2012-10-19 20:11:16 +0000 |
---|---|---|
committer | Shuxin Yang <shuxin.llvm@gmail.com> | 2012-10-19 20:11:16 +0000 |
commit | 970755e5197afb42a76a24a74cd6dc17721daf50 (patch) | |
tree | 6cfad3c8b2c649d34e247dfa22c2a1c73af9cdac /include/llvm/Intrinsics.td | |
parent | 9abfbdfc77efe715314f8ed673a7b6d25c00ca48 (diff) | |
download | llvm-970755e5197afb42a76a24a74cd6dc17721daf50.tar.gz llvm-970755e5197afb42a76a24a74cd6dc17721daf50.tar.bz2 llvm-970755e5197afb42a76a24a74cd6dc17721daf50.tar.xz |
This patch is to fix radar://8426430. It is about llvm support of __builtin_debugtrap()
which is supposed to consistently raise SIGTRAP across all systems. In contrast,
__builtin_trap() behave differently on different systems. e.g. it raises SIGTRAP on ARM, and
SIGILL on X86. The purpose of __builtin_debugtrap() is to consistently provide "trap"
functionality, in the mean time preserve the compatibility with on gcc on __builtin_trap().
The X86 backend is already able to handle debugtrap(). This patch is to:
1) make front-end recognize "__builtin_debugtrap()" (emboddied in the one-line change to Clang).
2) In DAG legalization phase, by default, "debugtrap" will be replaced with "trap", which
make the __builtin_debugtrap() "available" to all existing ports without the hassle of
changing their code.
3) If trap-function is specified (via -trap-func=xyz to llc), both __builtin_debugtrap() and
__builtin_trap() will be expanded into the function call of the specified trap function.
This behavior may need change in the future.
The provided testing-case is to make sure 2) and 3) are working for ARM port, and we
already have a testing case for x86.
git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@166300 91177308-0d34-0410-b5e6-96231b3b80d8
Diffstat (limited to 'include/llvm/Intrinsics.td')
-rw-r--r-- | include/llvm/Intrinsics.td | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/include/llvm/Intrinsics.td b/include/llvm/Intrinsics.td index 804db49556..5ccfde9e1a 100644 --- a/include/llvm/Intrinsics.td +++ b/include/llvm/Intrinsics.td @@ -420,7 +420,7 @@ def int_flt_rounds : Intrinsic<[llvm_i32_ty]>, GCCBuiltin<"__builtin_flt_rounds">; def int_trap : Intrinsic<[], [], [IntrNoReturn]>, GCCBuiltin<"__builtin_trap">; -def int_debugtrap : Intrinsic<[]>, +def int_debugtrap : Intrinsic<[], [], [IntrNoReturn]>, GCCBuiltin<"__builtin_debugtrap">; // NOP: calls/invokes to this intrinsic are removed by codegen |