Remove obsolete ??? comment in profile.c

Eric Botcazou ebotcazou@adacore.com
Wed Apr 6 06:18:00 GMT 2011


This is

	      location_t curr_location = single_succ_edge (bb)->goto_locus;
	      /* ??? The FILE/LINE API is inconsistent for these cases.  */
	      output_location (LOCATION_FILE (curr_location),
			       LOCATION_LINE (curr_location), &offset, bb);

which dates back to the pre-location-map days.  Moreover, the double call to 
expand_location hidden behind the LOCATION_FILE and LOCATION_LINE macros can 
be eliminated like a few lines above.

Tested on x86_64-suse-linux, applied on the mainline as obvious.


2011-04-06  Eric Botcazou  <ebotcazou@adacore.com>

	* profile.c (branch_prob): Move declaration of local variable.  Remove
	obsolete ??? comment.  Expand the location explicitly instead of using
	the LOCATION_FILE and LOCATION_LINE macros.


-- 
Eric Botcazou
-------------- next part --------------
A non-text attachment was scrubbed...
Name: p.diff
Type: text/x-diff
Size: 1327 bytes
Desc: not available
URL: <http://gcc.gnu.org/pipermail/gcc-patches/attachments/20110406/412b61b9/attachment.bin>


More information about the Gcc-patches mailing list