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

[Alex Bradbury via Phabricator via llvm-commits]

asb added inline comments.


================
Comment at: llvm/lib/Support/RISCVISAInfo.cpp:916
 
-static const char *ImpliedExtsF[] = {"zicsr"};
 static const char *ImpliedExtsD[] = {"f"};
+static const char *ImpliedExtsF[] = {"zicsr"};
----------------
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.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D148857