Stackmaps are used for OSR exits, which is a custom kind of unwinding. Hence, they
authorFilip Pizlo <fpizlo@apple.com>
Thu, 20 Feb 2014 23:57:31 +0000 (23:57 +0000)
committerFilip Pizlo <fpizlo@apple.com>
Thu, 20 Feb 2014 23:57:31 +0000 (23:57 +0000)
commit23ffb3ea10530e36f8b779570f8e1cc686708051
tree4291515fbec21d92799692b7eda5cae024cb2bcc
parent0aabe661a4315def30ddb2b4dbf290b513ffb747
Stackmaps are used for OSR exits, which is a custom kind of unwinding. Hence, they
should not be marked nounwind.

Marking them nounwind caused crashes in the WebKit FTL JIT, because if we enable
sufficient optimizations, LLVM starts eliding compact_unwind sections (or any unwind
data for that matter), making deoptimization via stackmaps impossible.

This changes the stackmap intrinsic to be may-throw, adds a test for exactly the
sympton that WebKit saw, and fixes TableGen to handle un-attributed intrinsics.

Thanks to atrick and philipreames for reviewing this.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@201826 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/IR/Intrinsics.td
unittests/ExecutionEngine/MCJIT/CMakeLists.txt
unittests/ExecutionEngine/MCJIT/MCJITCAPITest.cpp
unittests/ExecutionEngine/MCJIT/Makefile
utils/TableGen/IntrinsicEmitter.cpp