Re: [PATCH 1/6] kbuild: specify output names separately for each emission type from rustc
From: Miguel Ojeda
Date: Tue Jan 03 2023 - 15:45:12 EST
On Sat, Dec 31, 2022 at 7:42 AM Masahiro Yamada <masahiroy@xxxxxxxxxx> wrote:
>
> $ make -j$(nproc) samples/rust/rust_minimal.o samples/rust/rust_minimal.rsi \
> samples/rust/rust_minimal.s samples/rust/rust_minimal.ll
Yeah, we were testing the single targets, but not multiple at once, thanks!
> + --emit=dep-info=$(depfile) --emit=obj=$@ --emit=metadata=$(dir $@)$(patsubst %.o,lib%.rmeta,$(notdir $@)) \
Perhaps a newline here to avoid the lengthy line?
> hostc_flags = -Wp,-MMD,$(depfile) $(_hostc_flags)
> hostcxx_flags = -Wp,-MMD,$(depfile) $(_hostcxx_flags)
> -hostrust_flags = $(_hostrust_flags)
This was originally meant to be consistent with C and C++ indeed, but
if you prefer less variables, I guess it is fine, in which case,
should we update the C/C++ side too (in another series)?
Reviewed-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
Tested-by: Miguel Ojeda <ojeda@xxxxxxxxxx>
Cheers,
Miguel