[PATCH] D44897: [PowerPC] Do not emit record-form rotates when record-form andi/andis suffices

Hiroshi Inoue via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 12 10:21:38 PDT 2018


inouehrs added inline comments.


================
Comment at: lib/Target/PowerPC/PPCInstrInfo.cpp:1912
+      // same halfword.
+      int MBInHWord = MB >= 16 ? 1 : 0;
+      int MEInHWord = ME >= 16 ? 1 : 0;
----------------
MBInLoHWord/MEInLoHWord are more descriptive. I first think H means Hi before reading comment.
Also they can be bool, but it is ok if you prefer int for bool.



Repository:
  rL LLVM

https://reviews.llvm.org/D44897





More information about the llvm-commits mailing list