[PATCH, RFC, rs6000] PR80791 Consider doloop in ivopts

Jakub Jelinek jakub@redhat.com
Wed Apr 24 09:08:00 GMT 2019


On Wed, Apr 24, 2019 at 04:41:01PM +0800, Kewen.Lin wrote:
> gcc/ChangeLog

Not a review, just ChangeLog nits.

> 2019-04-24  Kewen Lin  <linkw@gcc.gnu.org>
> 
> 	PR middle-end/80791
> 	* target.def (predict_doloop_p): New.
> 	* targhooks.h (default_predict_doloop_p): New.
> 	* targhooks.c (default_predict_doloop_p): Likewise.
> 	* doc/tm.texi.in (TARGET_PREDICT_DOLOOP_P): New.
> 	* doc/tm.texi: Regenerate.
> 	* config/rs6000/rs6000.c (invalid_insn_for_doloop_p): New.
> 	  (costly_iter_for_doloop_p): New.
> 	  (rs6000_predict_doloop_p): New.

There should be no leading spaces after the tab on the lines that don't contain
a filename, so:
	* config/rs6000/rs6000.c (invalid_insn_for_doloop_p): New.
	(costly_iter_for_doloop_p): New.
	(rs6000_predict_doloop_p): New.
instead (and instead of saying just New. usually we write what kind of
new thing it is, so New function., New declaration. (or Declare.), New
macro. (or Define.), New method. etc.

> 	* gcc/expr.c (produce_memory_decl_rtl): New.

The gcc/ prefixes don't belong into gcc/ChangeLog, the filenames are always
relative to the ChangeLog file referencing those.

	Jakub



More information about the Gcc-patches mailing list