Re: [PATCH] RFC: Enable graphite at -O3 -fprofile_use

[Richard Biener <rguenther at suse dot de>]

Sorry for not replying earlier, it seems gmail eats messages sometimes
(thus also not replying to the mail directly).

You write

> Since graphite will not modify the CFG when it does not do any 
optimization,
> we would like to propose that graphite's polyhedral optimizer be enabled 
at
> -O3 -fprofile-use where, compile time is of lesser concern.

I think this is generally a good idea.  Note that for this we should
raise the minimum supported ISL version to one supporting the
timeout thingy and of course we need to make ISL a required
prerequesite for building GCC.  Which means making sure ISL builds
on all our primary and secondary host platforms (I think it does).

I'd also like to see runtime / compile-time (and code-size?) figures
for, say, SPEC CPU 2006 (with FDO of course).  Maybe also some
information on whether GRAPHITE did anything for any benchmark
in that suite (I notice GRAPHITE doesn't use the -fopt-info
machinery, aka dump_printf and friends with MSG_OPTIMIZED_LOCATIONS
or MSG_MISSED_OPTIMIZATIONS).

So please work on the above list of items.

Thanks,
Richard.