>>>> I definitely agree, which is why I added it in the first place :).
>>>> The problem is that it isn't hooked up with any of the other
>>>> diagnostics machinery, so it isn't controllable and doesn't give
>>>> decent location info (for example).
>>>>
>>>> I don't know that it really provides a good user experience.  In  
>>>> the
>>>> LTO case with debug info, we could do better.
>>>
>>> So what's your idea, detect it somewhere else?  Where were you
>>> thinking?
>>
>> For now I think we just just disable the warning.
>
> I'd prefer to wait until the replacement is in place, if you're OK
> with that.

I'm ok with that, but please reevaluate if it is causing chaos in the  
testsuite :)

-Chris
_______________________________________________
llvm-commits mailing list
llvm-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits

Reply via email to