Re: [C++ Patch] PR 80955 (Macros expanded in definition of user-defined literals)

[Paolo Carlini <paolo dot carlini at oracle dot com>]

Hi,

On 04/11/2017 23:37, Mukesh Kapoor wrote:
> I had included <inttypes.h> to get the definition of macro PRId64. I 
> have now modified the test case to remove all includes. I have added 
> the definition of the macro in the test case and also added 
> declarations of functions sprintf() strcmp(). I have attached the 
> revised patch. Thanks.
Excellent. I'm now committing the patch with a couple of additional 
tweaks to the testcase: 1- Since you carefully constructed it to return 
zero at run time if we do the right thing, you want a 'dg-do run' not a 
'dg-do compile'; 2- You also want a long ia64 variable, otherwise the 
testscase triggers a -Wformat warning.

Thanks,
Paolo.

////////////////////

Attachment: patch_80955_p
Description: Text document