I like the version without "function" in it; you might be returning from a
method.
On Fri, Dec 2, 2016 at 12:15 PM, Filipe Cabecinhas <
filcab+llvm.phabrica...@gmail.com> wrote:
> .Case("stack-use-after-return", "Use of returned stack memory")
>
>
> Maybe "Use of stack memory after (function) return"? (i couldn't decide
> whether to include "function". Either delete it or delete the parens.
> This is a very minor nit, so I'm ok with keeping the current wording if
> you prefer that.
>
>
> On Fri, 2 Dec 2016 at 20:04, Kuba (Brecka) Mracek via Phabricator <
> revi...@reviews.llvm.org> wrote:
>
>> kubabrecka added a comment.
>>
>>
>>
>> In https://reviews.llvm.org/D27017#611894, @filcab wrote:
>>
>>
>>
>> > LGTM
>>
>> >
>>
>> > (I commented on a minor nit. It might just be me, so feel free to keep
>> the current wording if you feel it's preferred)
>>
>>
>>
>>
>>
>> I'm not seeing this comment. Can you post it again?
>>
>>
>>
>>
>>
>> https://reviews.llvm.org/D27017
>>
>>
>>
>>
>>
>>
>>
>>
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
