Re: [PATCH] LoongArch: Enable initial Rust support

From: Miguel Ojeda
Date: Sun Jan 07 2024 - 09:29:47 EST


On Sat, Jan 6, 2024 at 8:04 AM WANG Rui <wangrui@xxxxxxxxxxx> wrote:
>
> ============ ================ ==============================================
> Architecture Level of support Constraints
> ============ ================ ==============================================
> +``loongarch`` Maintained

On top of what Huacai said, it would be nice to add another character
and realign the table so that `loongarch` it doesn't overflow (perhaps
best done in an additional, previous patch).

> + } else if cfg.has("LOONGARCH") {

Would it be possible to do it without a custom target in
`scripts/generate_rust_target.rs`? If it is not possible (e.g. there
is something that you cannot tweak otherwise on top of a built-in
target via the CLI flags), it is fine to use a custom target for the
moment.

For instance, this is how ARM is doing it:
https://lore.kernel.org/rust-for-linux/20231020155056.3495121-1-Jamie.Cunliffe@xxxxxxx/

Thanks!

Cheers,
Miguel