On Tue, Sep 13, 2016 at 02:38:54PM -0600, Jeff Law wrote:
> I believe a doc update of some kind is in order. With the doc update
> the entire series is OK.
Good catch, thanks. My tests finished, all results identical. I'll add
the following patch for the doc update (I hope the wording is strong enough).
Thanks,
Segher
2016-09-14 Segher Boessenkool <seg...@kernel.crashing.org>
* target.def (lra_p): Change commentary (for the manual) for the
new default.
* doc/tm.texi: Regenerate.
---
gcc/doc/tm.texi | 2 +-
gcc/target.def | 4 ++--
2 files changed, 3 insertions(+), 3 deletions(-)
diff --git a/gcc/doc/tm.texi b/gcc/doc/tm.texi
index dc5bcd6..0ca00d2 100644
--- a/gcc/doc/tm.texi
+++ b/gcc/doc/tm.texi
@@ -2861,7 +2861,7 @@ A target hook which can change allocno class for given
pseudo from
@end deftypefn
@deftypefn {Target Hook} bool TARGET_LRA_P (void)
-A target hook which returns true if we use LRA instead of reload pass. The
default version of this target hook returns always false, but new ports should
use LRA.
+A target hook which returns true if we use LRA instead of reload pass. The
default version of this target hook returns always true. New ports should use
LRA, and existing ports are encouraged to convert.
@end deftypefn
@deftypefn {Target Hook} int TARGET_REGISTER_PRIORITY (int)
diff --git a/gcc/target.def b/gcc/target.def
index 8509e7d..a4e4cbb 100644
--- a/gcc/target.def
+++ b/gcc/target.def
@@ -4929,8 +4929,8 @@ DEFHOOK
(lra_p,
"A target hook which returns true if we use LRA instead of reload pass.\
\
- The default version of this target hook returns always false, but new\
- ports should use LRA.",
+ The default version of this target hook returns always true. New ports\
+ should use LRA, and existing ports are encouraged to convert.",
bool, (void),
default_lra_p)
--
1.9.3
