================
@@ -4924,6 +4924,12 @@ def GetUnsafeStackPtr : Builtin {
   let Prototype = "void*()";
 }
 
+def UnsafeSigaltstack : Builtin {
+  let Spellings = ["__builtin_unsafe_sigaltstack"];
----------------
jakos-sec wrote:

Sounds good, I can create a safestack_interface.h in a separate PR.

I removed the builtin from this PR now and would:

1. Merge this PR as is (since it's not really usable for a user right now)
2. Create a PR creating `compiler-rt/include/sanitizer/safestack_interface.h` 
(with the existing builtins and `unsafe_sigaltstack()` and update SafeStack.rst 
telling people to use safestack_interface.h instead of the builtins.
3. (Optional?) Mark the builtins as deprecated
4. Once the stack of PRs is complete and `unsafe_sigaltstack()` is usable, 
adding it to SafeStack.rst and also updating the note that sigaltstack is not 
supported.



https://github.com/llvm/llvm-project/pull/196969
_______________________________________________
llvm-branch-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits

Reply via email to