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: Change prescan parameter to final_scan_insn from -2 to 0 in arc.c


Ian Lance Taylor <ian@airs.com> writes:

> An inspection of the code of final_scan_insn shows that the values -2
> and 0 for the prescan parameter are the same when HAVE_cc0 is not
> defined.  Every test of prescan is of the form (prescan > 0) or
> (prescan <= 0), except for one test guarded by #ifdef HAVE_cc0 which
> tests (prescan >= 0).
> 
> The ARC backend does not define HAVE_cc0.  There is code in arc.c
> which calls final_scan_insn with -2 for the prescan value.  This patch
> changes it to use 0 instead.
> 
> If this is approved, I plan to submit a followup patch to eliminate
> the prescan parameter entirely from final_scan_insn and final, as
> after this patch, prescan will always have the value 0.
> 
> Tested by rebuilding a cross-compiler and thinking about it.  I don't
> even know how to really test ARC code.  OK to commit?
> 
> Ian
> 
> 
> 2005-03-23  Ian Lance Taylor  <ian@airs.com>
> 
> 	* config/arc/arc.c (arc_output_function_epilogue): Pass prescan as
> 	0 when calling final_scan_insn.

Approved off list by RTH.  Committed.

Ian


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