[PATCH] D148857: [RISCV] NFC tweaks in extension descriptions, sorting

[Alex Bradbury via Phabricator via llvm-commits]

asb accepted this revision.
asb added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:916
 
-static const char *ImpliedExtsF[] = {"zicsr"};
 static const char *ImpliedExtsD[] = {"f"};
+static const char *ImpliedExtsF[] = {"zicsr"};
----------------
craig.topper wrote:
> asb wrote:
> > There is a canonical order for the single letter extensions (and for ordering those vs Z and X extensions). As such, I'd suggest:
> > * Single letter extensions in canonical order (i.e. F then D then V).
> > * Then a blank line, then the standard Z* extensions in alphabetical order
> > * Then a blank line, then the standard X* extensions in alphabetical order
> > 
> > I don't feel too strongly about this, so if you have a strongly different view I'd be OK with just plain alphabetic order. But let me know what you think.
> Alphabetical does make some sense here. These are all listed again in `ImpliedExts` which must me in alabetical order due to binary search.
Good point, that makes sense.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148857