[PATCH] add more detail to -Wconversion and -Woverflow (PR 80731)

[Martin Sebor]

On 06/02/2017 11:11 AM, Renlin Li wrote:
> Hi Martin,
>
> I noticed the following failures after your change r248431.
> FAIL: c-c++-common/Wfloat-conversion.c  -Wc++-compat   (test for
> warnings, line 42)
> FAIL: c-c++-common/Wfloat-conversion.c  -Wc++-compat   (test for
> warnings, line 43)
>
> It happens on arm target which is not a large_long_double target.
> The patch here add the missing target selector. After the change, those
> test
> won't checked in arm target.
>
> Here I have a simple fix to it. Okay to commit?

r248431 wasn't meant to change when any of these warnings fire,
only their text and the amount of detail included in them.  The
removal of the { target large_long_double } selector was by
accident.

Please go ahead and commit your fix.  Thanks!

>
> gcc/testsuite/ChangeLog:
>
> 2017-06-02 Renlin Li <renlin.li@arm.com>
>
> * c-c++-common/Wfloat-conversion.c: Add large_long_double target
> selector to related line.
>
>
>
> And there is another failure:
> FAIL: gcc.dg/utf16-4.c  (test for warnings, line 15)
>
> The warning message is slightly different from expected.
> utf16-4.c:10:15: warning: character constant too long for its type
> utf16-4.c:15:15: warning: conversion from 'long unsigned int' to
> 'char16_t {aka short unsigned int}' changes value from '410401' to '17185'

The test passes for me now.  The initial commit had introduced
bug 80731 but it's recently been fixed.  Can you please try
again with the latest sources?

Martin