Re: [PATCH v6 4/4] RISC-V: handle extension configs for bindgen, re-enable gcc + rust builds
From: Asuna Yang
Date: Fri Feb 06 2026 - 07:15:54 EST
On 1/29/26 21:49, Charalampos Mitrodimas wrote:
I'm under the impression that the `!RUST ||` guard here doesn't
actually prevent the `$(bindgen-backend-option,...)` call from being
executed. `$(...)` shell expansions should happen during the textual
substitution phase, before symbol dependency evaluation occurs, check
documentation at kconfig-macro-language.rst lines 228-229.
I suppose this is unavoidable if we decide to probe options rather than rely on version numbers.
Similar issue also occurs in `cc-option`, for example:
depends on !64BIT || $(cc-option,-mabi=lp64 -march=rv64imv)
depends on !32BIT || $(cc-option,-mabi=ilp32 -march=rv32imv)
For 64-bit builds, `-march=rv32imv` will still be probed; for 32-bit builds, `-march=rv64imv` will still be probed.
I agree with Nathan that this trivial waste is worth it for the sake of simplicity.
Best regards,
Asuna