On 12/06/2017 12:58 PM, Martin Sebor wrote:
> On 12/06/2017 12:41 PM, Jeff Law wrote:
>> On 12/06/2017 12:27 PM, Martin Sebor wrote:
>>> For reference, I committed r255450 to avoid the test failure
>>> reported on the arm-none-eabi target.  The warning is valid
>>> but unrelated to the purpose of the test so I pruned it from
>>> its output.
>>>
>>> Martin
>>>
>>> Index: gcc/testsuite/ChangeLog
>>> ===================================================================
>>> --- gcc/testsuite/ChangeLog    (revision 255449)
>>> +++ gcc/testsuite/ChangeLog    (working copy)
>>> @@ -9,6 +9,9 @@
>>>
>>>  2017-12-06  Martin Sebor  <mse...@redhat.com>
>>>
>>> +    PR testsuite/83303
>>> +    * g++.dg/opt/new1.C: Prune warning from test output.
>> The path to the erroneous behavior should be caught and eliminated by
>> DOM.  That's the whole point behind one of the recent changes.
>>
>> Instead we need to figure out why it wasn't caught on arm-none-eabi.
> 
> Yes, I should have suggested to look into the failure to do
> that, independently of the test.  I can open a new bug for
> it unless it's already on your to-do list.
I think the right thing to do is revert the change and keep 83303 open
until we see what's different about arm-none-eabi.  I only noticed this
because I'd debugged that test on x86_64 and fixed DOM to eliminate the
problem path over the weekend...  So something is clearly different on
that platform.

jeff

Reply via email to