[PATCH] Add a "probe-stack" attribute
John Kåre Alsaker
john.mailinglists at gmail.com
Sat Jan 17 00:57:59 PST 2015
I have a patch to add __probestack to compiler-rt. No other such
function have a configurable name, so I don't see why __probestack
should be different.
I'm not sure how to test for #3 as I don't think any calling
convention has RAX as live-in.
I don't know if #4 even handles the edge case(s), which is why I added
an assertion, so I don't know how to test for those. I can add a test
for the non-edge case though.
The stack-probe-size attribute interacts as expected (it just controls
the stack allocation size for which probes are required).
On Fri, Jan 16, 2015 at 10:30 PM, Reid Kleckner <rnk at google.com> wrote:
> On Fri, Jan 16, 2015 at 12:43 PM, David Majnemer <david.majnemer at gmail.com>
>> Reid, I'm not so sure.
>> We already have a probe-stack-size attribute, how does this attribute
>> interact with that one? Which targets have a __probestack that can be
> There aren't any targets that use __probestack. It's new this patch. Maybe
> the attribute should take the symbol name as a value so it's user
More information about the llvm-commits