[PATCH] D45284: [RISCV] More validations on the input value of -march=

Alex Bradbury via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Apr 19 05:50:21 PDT 2018


asb added a comment.
Herald added a subscriber: edward-jones.

This is looking great, the only remaining code comment I have is that getExtensionFeatures needs a comment describing it.

The remaining issue I have is more of a spec issue - do canonical ordering requirements apply to extension categories? e.g. is sfoo_xbar illegal because it should be ordered xbar_sfoo? From my reading of the note below table 22.1 in the spec I think it is illegal and ordering of non-standard user-level vs standard supervisor-level vs nons-standard supervisor-level should be checked. Do you agree with my interpretation?


https://reviews.llvm.org/D45284





More information about the cfe-commits mailing list