[PATCH] D29424: [PPC] WIP - Add statistics for mem* fucntions

Sean Fertile via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 2 06:50:36 PST 2017


sfertile added inline comments.


================
Comment at: lib/CodeGen/SelectionDAG/SelectionDAG.cpp:119
+    *Unaligned += 1;
+  else if (Align == 16)
+    *VectorAligned += 1;
----------------
inouehrs wrote:
> Is `if (Align % 16 == 0)` better to support Align = 32, 64, ... ?
Your right, I'll change this to Align >= 16, I don't necessarily care that its exactly 16 byte aligned, rather I'm interested in it being at least 16 byte aligned.


Repository:
  rL LLVM

https://reviews.llvm.org/D29424





More information about the llvm-commits mailing list