[email protected]
[Top] [All Lists]

Re: [patch]: Add push_macro/pop_macro feature for all targets by movin

Subject: Re: [patch]: Add push_macro/pop_macro feature for all targets by moving it from C frontent into libcpp
From: Kai Tietz
Date: Tue, 3 Nov 2009 18:09:05 +0100
2009/11/3 Joseph S. Myers <[email protected]>:
> On Tue, 3 Nov 2009, Kai Tietz wrote:
>> Hello,
>> the current implementation of the push_macro/pop_macro feature for
>> windows targets was broken for C++ frontend and preprocessor. Just for
>> C it was working proper. So I would like to move this feature from C
>> frontend into libcpp as internal support pragma for all targets.
> I don't see a testcase added to the testsuite by the patch.  You need to
> add at least one that fails before and passes after the patch, unless
> there is already one there.
> (This is not a review of the rest of the patch.)
> --
> Joseph S. Myers
> [email protected]

Ok. Revised version with additional testcase.

2009-11-03  Kai Tietz  <[email protected]>

        * gcc.c-torture/execute/pushpop_macro.c: New testcase.


|  (\_/) This is Bunny. Copy and paste
| (='.'=) Bunny into your signature to help
| (")_(") him gain world domination
<Prev in Thread] Current Thread [Next in Thread>