This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH to use cpplib's source_location for location_t
- From: "Per Bothner" <per at bothner dot com>
- To: "Aaron W. LaFramboise" <aaronavay62 at aaronwl dot com>
- Cc: "Gcc Patch List" <gcc-patches at gcc dot gnu dot org>
- Date: Thu, 1 Jul 2004 00:13:12 -0700 (PDT)
- Subject: Re: PATCH to use cpplib's source_location for location_t
- References: <40E093CC.40802@bothner.com> <20040628222114.GB12535@redhat.com> <40E2ECED.4080307@bothner.com> <40E38881.7040806@aaronwl.com>
Aaron W. LaFramboise wrote:
> Per Bothner wrote:
>
>>Index: final.c
>>=================================================================== RCS
file: /cvs/gcc/gcc/gcc/final.c,v
>>retrieving revision 1.315
>>diff -u -p -r1.315 final.c
>>--- final.c 15 Jun 2004 18:02:19 -0000 1.315
>>+++ final.c 30 Jun 2004 16:25:50 -0000
>>@@ -1520,8 +1520,13 @@ final (rtx first, FILE *file, int optimi
>> if (GET_CODE (insn) == NOTE && NOTE_LINE_NUMBER (insn) > 0)
>> {
>> if (last != 0
>>+#ifdef USE_MAPPED_LOCATION
>>+ && NOTE_SOURCE_LOCATION (insn) == NOTE_SOURCE_LOCATION (last) +#else
>> && NOTE_LINE_NUMBER (insn) == NOTE_LINE_NUMBER (last)
>>- && NOTE_SOURCE_FILE (insn) == NOTE_SOURCE_FILE (last))
>>+ && NOTE_SOURCE_FILE (insn) == NOTE_SOURCE_FILE (last)
>>+#endif)
>>+ )
>> {
>> delete_insn (insn); /* Use delete_note. */
>> continue;
>
>
> I don't think the ) is supposed to be after #endif. HEAD compiles fine
after I remove it.
That doesn't make sense ... I'm fairly sure the ')' is needed.
Both visual inspection, and the compiler agree. Perhaps you're
confused about the patch syntax?
--
--Per Bothner
per@bothner.com http://per.bothner.com/