This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: PATCH to use cpplib's source_location for location_t


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/





Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]