[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