[PATCH] D86201: [GlobalISel] Fix and tidy up documentation for ValueMapping class (NFC)

Bjorn Pettersson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 01:02:27 PDT 2020


bjope added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/GlobalISel/RegisterBankInfo.h:107
   /// PartialMapping[] = {
-  /// /*32-bit add*/    {0, 32, GPR}, // Scalar entry repeated for first vec elt.
-  /// /*2x32-bit add*/  {0, 32, GPR}, {32, 32, GPR},
-  /// /*<2x32-bit> vadd {0, 64, VPR}
+  /// /*32-bit add*/      {0, 32, GPR}, // Scalar entry repeated for first vec elt.
+  /// /*2x32-bit add*/    {0, 32, GPR}, {32, 32, GPR},
----------------
A bit unfortunate to get too long lines here. Not sure if that is worse compared to things not being aligned in the code comments.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86201



More information about the llvm-commits mailing list