[llvm] r292188 - [TLI] Add prototype checking for all remaining LibFuncs.

[Ahmed Bougacha via llvm-commits]

On Tue, Jan 31, 2017 at 9:20 AM, Hans Wennborg <hans at chromium.org> wrote:
> On Mon, Jan 30, 2017 at 12:48 PM, Friedman, Eli <efriedma at codeaurora.org> wrote:
>> On 1/27/2017 7:16 PM, Ahmed Bougacha wrote:
>>>
>>> Hey folks,
>>>
>>> I'm pondering whether to merge this to 4.0.  It's not a small change,
>>> but it's hopefully only making existing functionality stricter.
>>>
>>> It fixes a handful of issues, some introduced by r267758 (in 3.9),
>>> some of which can be exposed in C code (using invalid libc protos; I
>>> could probably reduce the patch to the funcs I know of?).
>>>
>>> Does that sound desirable? Thanks!
>>
>>
>> The argument in favor is that this is a bugfix for an issue which could
>> result in weird crashes or miscompiles.  The argument against is that this
>> is a large patch, the issue isn't a regression from 3.9, the problem doesn't
>> reproduce with well-formed code, and this problem hasn't shown up in
>> real-world code.  (Feel free to correct me if one of those statements is
>> wrong.)
>>
>> Given that, I don't think I would suggest merging this to 4.0.

This makes sense to me; let's hold off on this.  Thanks for the advice people!

-Ahmed