]> git.ipfire.org Git - thirdparty/gcc.git/commitdiff
lto: reimplement print_lto_docs_link [PR116613]
authorDavid Malcolm <dmalcolm@redhat.com>
Thu, 10 Oct 2024 01:26:08 +0000 (21:26 -0400)
committerDavid Malcolm <dmalcolm@redhat.com>
Thu, 10 Oct 2024 01:26:08 +0000 (21:26 -0400)
gcc/ChangeLog:
PR other/116613
* lto-wrapper.cc (print_lto_docs_link): Use a format string rather
than building the string manually.  Fix memory leak of "url" by
using label_text.

Signed-off-by: David Malcolm <dmalcolm@redhat.com>
gcc/lto-wrapper.cc

index 141856c9b67b5110e55241d6e7d2bb5caccad683..9d7fe18b2296774bcb8205ecd1a1aac55713d62c 100644 (file)
@@ -1366,19 +1366,10 @@ init_num_threads (void)
 void
 print_lto_docs_link ()
 {
-  bool print_url = global_dc->m_printer->supports_urls_p ();
-  const char *url = global_dc->make_option_url (OPT_flto);
-
-  pretty_printer pp;
-  pp.set_url_format (URL_FORMAT_DEFAULT);
-  pp_string (&pp, "see the ");
-  if (print_url)
-    pp_begin_url (&pp, url);
-  pp_string (&pp, "%<-flto%> option documentation");
-  if (print_url)
-    pp_end_url (&pp);
-  pp_string (&pp, " for more information");
-  inform (UNKNOWN_LOCATION, pp_formatted_text (&pp));
+  label_text url = label_text::take (global_dc->make_option_url (OPT_flto));
+  inform (UNKNOWN_LOCATION,
+         "see the %{%<-flto%> option documentation%} for more information",
+         url.get ());
 }
 
 /* Test that a make command is present and working, return true if so.  */