From: Arthur Cohen <arthur.cohen@embecosm.com>
To: Thomas Schwinge <thomas@codesourcery.com>,
Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
Cc: gcc-rust@gcc.gnu.org, gcc-patches@gcc.gnu.org,
Richard Biener <richard.guenther@gmail.com>
Subject: Re: [PATCH v2 3/4] build: Add libgrust as compilation modules
Date: Thu, 26 Oct 2023 14:07:01 +0200 [thread overview]
Message-ID: <a12cd82f-3eec-4b41-887a-fd029054aa17@embecosm.com> (raw)
In-Reply-To: <87sf5y1iuf.fsf@euler.schwinge.homeip.net>
Hi Thomas,
On 10/25/23 23:40, Thomas Schwinge wrote:
> Hi!
>
> On 2023-10-25T13:06:48+0200, Arthur Cohen <arthur.cohen@embecosm.com> wrote:
>> From: Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
>>
>> Define the libgrust directory as a host compilation module as well as
>> for targets.
>
> I don't see a response to Richard's comments:
> <https://inbox.sourceware.org/CAFiYyc0Pcqfbknt06FaMMsNrf7Ww4Kbor6QVeTWj++KmUP1nOw@mail.gmail.com>.
> Re "doesn't build libgrust if [Rust is not enabled]", I suppose (but have
> not checked) this works for the *target* libgrust module via
> 'gcc/rust/config-lang.in:target_libs' requesting 'target-libgrust' only
> if the Rust language is enabled? I don't know what enables/disables the
> *host* libgrust build?
>
>
> My comments:
> <https://inbox.sourceware.org/871qek608j.fsf@euler.schwinge.homeip.net>
> don't seem to have been addressed?
Sorry about that! The fixup commits you mentioned should have been
integrated already to the commits I pushed. I seem to have messed
something up in my branch, as your last comment should have been
addressed too - I'll resend clean commits where all of this is checked.
>
>
> Also, don't you first have to get in "build: Regenerate build files"
> before the 'gcc/rust/config-lang.in:target_libs' change, to avoid
> breaking bisection?
>
> Maybe, after all, don't have a separate "build: Regenerate build files"
> commit, but instead do the regeneration as part of the commits adding the
> source files, in usual GCC style?
Alright, that sounds good.
>
> And then, the 'contrib/gcc_update' change that's currently in
> "libgrust: Add entry for maintainers and stub changelog file" should be
> part of the commit that actually adds the files referenced therein.
Okay!
Thanks for the review. I'll address these shortly.
>
>
> Grüße
> Thomas
>
>
>> ChangeLog:
>>
>> * Makefile.def: Add libgrust as host & target module.
>> * configure.ac: Add libgrust to host tools list.
>>
>> gcc/rust/ChangeLog:
>>
>> * config-lang.in: Add libgrust as a target module for the rust
>> language.
>>
>> Signed-off-by: Pierre-Emmanuel Patry <pierre-emmanuel.patry@embecosm.com>
>> ---
>> Makefile.def | 2 ++
>> configure.ac | 3 ++-
>> gcc/rust/config-lang.in | 2 ++
>> 3 files changed, 6 insertions(+), 1 deletion(-)
>>
>> diff --git a/Makefile.def b/Makefile.def
>> index 15c068e4ac4..929a6f0a08e 100644
>> --- a/Makefile.def
>> +++ b/Makefile.def
>> @@ -149,6 +149,7 @@ host_modules= { module= libcc1; extra_configure_flags=--enable-shared; };
>> host_modules= { module= gotools; };
>> host_modules= { module= libctf; bootstrap=true; };
>> host_modules= { module= libsframe; bootstrap=true; };
>> +host_modules= { module= libgrust; };
>>
>> target_modules = { module= libstdc++-v3;
>> bootstrap=true;
>> @@ -192,6 +193,7 @@ target_modules = { module= libgm2; lib_path=.libs; };
>> target_modules = { module= libgomp; bootstrap= true; lib_path=.libs; };
>> target_modules = { module= libitm; lib_path=.libs; };
>> target_modules = { module= libatomic; bootstrap=true; lib_path=.libs; };
>> +target_modules = { module= libgrust; };
>>
>> // These are (some of) the make targets to be done in each subdirectory.
>> // Not all; these are the ones which don't have special options.
>> diff --git a/configure.ac b/configure.ac
>> index 692dc716343..b2a5511bab1 100644
>> --- a/configure.ac
>> +++ b/configure.ac
>> @@ -133,7 +133,7 @@ build_tools="build-texinfo build-flex build-bison build-m4 build-fixincludes"
>>
>> # these libraries are used by various programs built for the host environment
>> #f
>> -host_libs="intl libiberty opcodes bfd readline tcl tk itcl libgui zlib libbacktrace libcpp libcody libdecnumber gmp mpfr mpc isl libiconv libctf libsframe"
>> +host_libs="intl libiberty opcodes bfd readline tcl tk itcl libgui zlib libbacktrace libcpp libcody libdecnumber gmp mpfr mpc isl libiconv libctf libsframe libgrust "
>>
>> # these tools are built for the host environment
>> # Note, the powerpc-eabi build depends on sim occurring before gdb in order to
>> @@ -164,6 +164,7 @@ target_libraries="target-libgcc \
>> target-libada \
>> target-libgm2 \
>> target-libgo \
>> + target-libgrust \
>> target-libphobos \
>> target-zlib"
>>
>> diff --git a/gcc/rust/config-lang.in b/gcc/rust/config-lang.in
>> index aac66c9b962..8f071dcb0bf 100644
>> --- a/gcc/rust/config-lang.in
>> +++ b/gcc/rust/config-lang.in
>> @@ -29,4 +29,6 @@ compilers="rust1\$(exeext)"
>>
>> build_by_default="no"
>>
>> +target_libs="target-libffi target-libbacktrace target-libgrust"
>> +
>> gtfiles="\$(srcdir)/rust/rust-lang.cc"
>> --
>> 2.42.0
> -----------------
> Siemens Electronic Design Automation GmbH; Anschrift: Arnulfstraße 201, 80634 München; Gesellschaft mit beschränkter Haftung; Geschäftsführer: Thomas Heurung, Frank Thürauf; Sitz der Gesellschaft: München; Registergericht München, HRB 106955
next prev parent reply other threads:[~2023-10-26 10:07 UTC|newest]
Thread overview: 16+ messages / expand[flat|nested] mbox.gz Atom feed top
2023-10-25 11:06 [PATCH v2 1/4] libgrust: Add entry for maintainers and stub changelog file Arthur Cohen
2023-10-25 11:06 ` [PATCH v2 2/4] libgrust: Add libproc_macro and build system Arthur Cohen
2023-10-26 8:21 ` Thomas Schwinge
2023-10-27 7:51 ` Iain Sandoe
2023-10-27 14:20 ` Thomas Schwinge
2023-10-27 20:32 ` Disable target libgrust if we're not building target libstdc++ (was: [PATCH v2 2/4] libgrust: Add libproc_macro and build system) Thomas Schwinge
2023-12-18 16:58 ` libgrust: 'AM_ENABLE_MULTILIB' only for target builds [PR113056] " Thomas Schwinge
2024-01-04 15:43 ` libgrust: 'AM_ENABLE_MULTILIB' only for target builds [PR113056] Pierre-Emmanuel Patry
2023-12-15 9:53 ` [PATCH v2 2/4] libgrust: Add libproc_macro and build system Thomas Schwinge
2024-01-16 15:07 ` Jason Merrill
2023-10-25 11:06 ` [PATCH v2 3/4] build: Add libgrust as compilation modules Arthur Cohen
2023-10-25 11:09 ` [PATCH v2 4/4] build: Regenerate build files Arthur Cohen
2023-10-25 21:40 ` [PATCH v2 3/4] build: Add libgrust as compilation modules Thomas Schwinge
2023-10-26 12:07 ` Arthur Cohen [this message]
2023-10-27 10:24 ` Only build host libgrust if the Rust language is enabled (was: [PATCH v2 3/4] build: Add libgrust as compilation modules) Thomas Schwinge
2023-10-25 11:06 ` [PATCH v2 4/4] build: Regenerate build files Arthur Cohen
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=a12cd82f-3eec-4b41-887a-fd029054aa17@embecosm.com \
--to=arthur.cohen@embecosm.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=gcc-rust@gcc.gnu.org \
--cc=pierre-emmanuel.patry@embecosm.com \
--cc=richard.guenther@gmail.com \
--cc=thomas@codesourcery.com \
/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).