[PATCH] D143953: [RISCV] Accept zicsr and zifencei command line options

[Philip Reames via Phabricator via cfe-commits]

reames added a comment.

@jrtc27 Not sure if this changes your take, but I realized the variant being introduced is maybe much less interesting than I'd first thought.  We generally make no effort to make sure an extension was defined in the spec version corresponding to our base revision.  Given that, we have a bunch of cases where we allow I2.0 + some random extension.  Given that, this one stops looking all that interesting.  It doesn't actually set much precedent - because we already did that, a long time ago.

If you agree with that framing, I'll rework the description.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143953