[llvm] r290024 - [Hexagon] Fix some Clang-tidy modernize and Include What You Use warnings; other minor fixes (NFC).

Eugene Zelenko via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 19 07:14:10 PST 2016


Hi, Malcolm!

On Mon, Dec 19, 2016 at 5:04 AM, Malcolm Parsons
<malcolm.parsons at gmail.com> 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
>
> --
> Malcolm Parsons

Thank you for help! And sorry for all troubles.

Eugene.


More information about the llvm-commits mailing list