[llvm] r290024 - [Hexagon] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).
Malcolm Parsons via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 19 05:53:03 PST 2016
On 19 December 2016 at 13:06, Krzysztof Parzyszek via llvm-commits
<llvm-commits at lists.llvm.org> wrote:
> On 12/19/2016 7:04 AM, Malcolm Parsons via llvm-commits wrote:
>>
>> On 18 December 2016 at 15:56, Eugene Zelenko <eugene.zelenko at gmail.com>
>> wrote:
>>>
>>> On Sun, Dec 18, 2016 at 7:47 AM, Malcolm Parsons
>>> <malcolm.parsons at gmail.com> wrote:
>>>>
>>>> On 17 December 2016 at 01:09, Eugene Zelenko via llvm-commits
>>>> <llvm-commits at lists.llvm.org> wrote:
>>>>>
>>>>> - if (P == INT_MIN)
>>>>> + if (P == std::numeric_limits<int>::max())
>>>>
>>>>
>>>> MIN -> max?
>>>
>>>
>>> I'm very sorry to introduce problem. And thank you for spotting it!
>>>
>>> But could not fix it until next year. Could you please commit this
>>> change?
>>
>>
>> r290100
>>
> How was this problem introduced? Was this a manual change, or is it a
> problem with some tool?
I'm guessing that Eugene changed it manually.
clang-tidy and IWYU don't make this change.
--
Malcolm Parsons
More information about the llvm-commits
mailing list