From: Alexandre Oliva <oliva@adacore.com>
To: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH] Support multilib-aware target lib flags self-specs overriding
Date: Thu, 06 Oct 2022 05:24:38 -0300 [thread overview]
Message-ID: <orr0zltl4p.fsf@lxoliva.fsfla.org> (raw)
In-Reply-To: <ortu84x69l.fsf@lxoliva.fsfla.org> (Alexandre Oliva's message of "Tue, 28 Jun 2022 10:32:54 -0300")
On Jun 28, 2022, Alexandre Oliva <oliva@adacore.com> wrote:
> Support multilib-aware target lib flags self-specs overriding
> This patch introduces -fmultiflags, short for multilib TFLAGS, as an
> option that does nothing by default, but that can be added to TFLAGS
> and mapped to useful options by driver self-specs.
> Regstrapped on x86_64-linux-gnu. Posted mainly FTR, but... I wouldn't
> mind putting it in, so... Ok to install? ;-)
Since this patch didn't seem to gain much traction towards acceptance,
I've been thinking of other ways to accomplish its goal, namely, to
enable target libs to be built with additional flags, like setting
TFLAGS, but enabling different multilibs to be built with different
flags.
The other way that came to mind was to add settings to the multilib
target fragment configurations, say MULTILIB_TFLAGS, to map multilibs to
extra flags to use, and then adjust the target lib multilib flag build
machinery to extract and add these flags when configuring and building
each library. Yuck.
I'd much rather we could use -fmultiflags, a far more elegant
arrangement IMHO, so...
Ping? https://gcc.gnu.org/pipermail/gcc-patches/2022-June/597419.html
> for gcc/ChangeLog
> * common.opt (fmultiflags): New.
> * doc/invoke.texi: Document it.
> * gcc.cc (driver_self_specs): Discard it.
> * opts.cc (common_handle_option): Ignore it in the driver.
Since this is kind of a build machinery feature, would anyone mind if I
self-approved this with my build machinery maintainer hat on?
Please raise your objection within one week if you do.
Thanks,
--
Alexandre Oliva, happy hacker https://FSFLA.org/blogs/lxo/
Free Software Activist GNU Toolchain Engineer
Disinformation flourishes because many people care deeply about injustice
but very few check the facts. Ask me about <https://stallmansupport.org>
next prev parent reply other threads:[~2022-10-06 8:24 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2022-05-20 19:13 Alexandre Oliva
2022-05-28 7:02 ` Alexandre Oliva
2022-06-01 22:00 ` Hans-Peter Nilsson
2022-06-03 7:15 ` Alexandre Oliva
2022-06-28 13:32 ` Alexandre Oliva
2022-10-06 8:24 ` Alexandre Oliva [this message]
2022-11-05 9:23 ` Alexandre Oliva
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=orr0zltl4p.fsf@lxoliva.fsfla.org \
--to=oliva@adacore.com \
--cc=gcc-patches@gcc.gnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).