[PATCH] Implement __builtin_stack_pointer

Tim Northover t.p.northover at gmail.com
Wed Mar 26 04:09:23 PDT 2014


  Hi Renato,

  The code changes look innocuous enough, but I think the documentation may be a bit over-enthusiastic in its guarantees:


================
Comment at: docs/LangRef.rst:6802-6803
@@ +6801,4 @@
+
+The '``llvm.stackpointer``' intrinsic returns the current value of the
+stack pointer register.
+
----------------
Isn't that a bit strong? I think things like VLAs can cause sp to change in the middle of a function and I dread to think what it'll return in those cases.


http://llvm-reviews.chandlerc.com/D3184



More information about the llvm-commits mailing list