Skip to content

Conversation

@koblonczek
Copy link

When no target (and neither custom target) was specified, it defaulted to rv32imc which overrode any isa and mabi supplied as arguments. This prevented setting any custom isa and mabi from the command line. New behavior is to only override these if they aren't set by the user explicitly (values specified with --isa and --mabi options override --target).

Copy link
Collaborator

@tmichalak tmichalak left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

When no target (and neither custom target) was specified, it defaulted to rv32imc
which overrode any isa and mabi supplied as arguments. This prevented setting any
custom isa and mabi from the command line. New behavior is to only override these
if they aren't set by the user explicitly (values specified with --isa and --mabi
options override --target).

Signed-off-by: Krzysztof Obłonczek <[email protected]>
@wkkuna wkkuna force-pushed the kobl/fix-isa-arg-handling branch from 4a08bd7 to 5b2042a Compare February 7, 2025 08:25
@wkkuna wkkuna changed the base branch from master to veer-el2 February 7, 2025 08:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants