[PATCH] Internal option to warn on a given stack size.

Sean Silva silvas at purdue.edu
Tue Jun 4 12:22:44 PDT 2013


On Mon, Jun 3, 2013 at 8:26 PM, Quentin Colombet <qcolombet at apple.com>wrote:

> Hi Sean,
>
> Thanks for the pointer!
>
> It would be sufficient, but it seems to require a large amount of work for
> this to be done properly.
> See r174748.
>
> In my opinion, the proposed patch can be seen as the first step towards
> this warning. When the proper diagnostic framework land in LLVM, this could
> be turned into the desired warning.
>
> What do you think?
>
>
I think it is a reasonable first step, but as Evan points out, it is really
more of a hack (dumping to errs() :| )than a proper solution. What are the
next steps to evolving this feature into a proper solution, and are you
guys planning on taking those steps? I don't think we want to just
introduce a hack and leave it as such.

-- Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130604/beae2467/attachment.html>


More information about the llvm-commits mailing list