[PATCH] D115921: [RISCV] Refactor the RISCV ISA extension info and target features to support multiple extension version

Luís Marques via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Sat Dec 18 06:08:38 PST 2021


luismarques added a comment.

> enable 'm' extension with passing mattr=+m After this patch, it would be -mattr=+m2p0.

It's not obvious to me that support for extension versions should mean or has to mean that we always explicitly specify the version. Why can't we keep supporting the option `mattr=+m`, which would be mapped to `mattr=+m,+m2p0`, or whatever the current default `m` version happens to be?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D115921/new/

https://reviews.llvm.org/D115921



More information about the cfe-commits mailing list