[PATCH] D34386: [PATCH 1/2] Add a "probe-stack" attribute (revised)

David Majnemer via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 20 11:15:00 PDT 2017


majnemer added inline comments.


================
Comment at: docs/LangRef.rst:1478-1480
+    If a function that has a ``"probe-stack"`` attribute is inlined into a
+    function that doesn't have a ``"probe-stack"`` attribute, then the
+    resulting function will have a ``"probe-stack"`` attribute.
----------------
What if both have a probe-stack attribute but they disagree? Should we disallow inlining? Should we ignore the callee attribute?


https://reviews.llvm.org/D34386





More information about the llvm-commits mailing list