[PATCH] D48602: `llvm.experimental.stackmap` is erroneously marked `Throws`?

[Edd Barrett via Phabricator via llvm-commits]

vext01 added a comment.

Hi @reames ,

Thanks for looking into this. I *think* I understand.

Another bug which stopped me from using stackmaps is:
https://bugs.llvm.org/show_bug.cgi?id=38277

Do you know if anyone is actually using the stackmap functionality? As far as I know JavaScriptCore used to use it, but have since moved on. It worked for them because they had only one compilation unit, but as far as I can see, stackmaps do not generalise to >1 CU, making them very limited.

I wonder if stackmap support should be removed, or at least generalised for >1CU?


Repository:
  rL LLVM

https://reviews.llvm.org/D48602