Re: Warning policy?

["Kaveh R. Ghazi" <ghazi at caip dot rutgers dot edu>]

 > From: Andreas Schwab <schwab@issan.informatik.uni-dortmund.de>
 > 
 > "Kaveh R. Ghazi" <ghazi@caip.rutgers.edu> writes:
 > 
 > |> 	The "label `???' defined but not used" warnings mostly appear in
 > |> insn-recog.c.  In this case genrecog.c needs to be fixed.  It may be
 > |> non-trivial to deduce which labels are unnecessary, I haven't looked. 
 > |> Another approach might be to get the compiler to allow attribute
 > |> __unused__ on a label and just mark all of them unused in the generated
 > |> insn-recog.c file.
 > 
 > Well, the brute force method would be to just emit a `goto L' before
 > each label L.
 > -- 
 > Andreas Schwab                                      "And now for something


	Sounds like a good idea.

The comments in jump.c seem to indicate that the jump pass will optimize
away these gotos.  I.e. it says it "deletes jumps that jump to the
following insn" which I believe this case fits.  So we don't pay
a penalty for this hack.

	I'll submit a patch after the freeze is lifted.

		Thanks,
		--Kaveh
--
Kaveh R. Ghazi			Engagement Manager / Project Services
ghazi@caip.rutgers.edu		Icon CMT Corp.