luismarques added a comment.

Possibly we want to add this change to the release notes?



================
Comment at: llvm/lib/Target/RISCV/RISCVFeatures.td:304-308
+                       "'Zcb' (Shortened format for basic bit manipulation 
instructions)", 
+                       [FeatureExtZca]>;
+def HasStdExtZcb : Predicate<"Subtarget->hasStdExtZcb()">,
+                             AssemblerPredicate<(all_of FeatureExtZcb),
+                             "'Zcb' (Shortened format for basic bit 
manipulation instructions)">;
----------------
Is there any particular reason why this is described as a "shortened format" 
instead of the more common "compressed"?


================
Comment at: llvm/lib/Target/RISCV/RISCVInstrInfoZc.td:9-10
+///
+/// This file describes the RISC-V instructions from the 'Zc' Code-size 
+/// reduction extension, version 0.70.4.
+/// This version is still experimental as the 'Zc' extension hasn't been
----------------
Ditto "Code-size reduction" -> "compressed"?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131141

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to