[PATCH] D28637: [PPC] Inline expansion of memcmp

[Tony Jiang via Phabricator via llvm-commits]

jtony added inline comments.


================
Comment at: include/llvm/Target/TargetLowering.h:1002
+  /// return the limit for functions that have OptSize attribute.
+  unsigned getMaxLoadsPerMemcmp(bool OptSize) const {
+    return OptSize ? MaxLoadsPerMemcmpOptSize : MaxLoadsPerMemcmp;
----------------
This variable name sounds like an integer rather than a bool. But since this is NOT newly introduced (historical legacy  actually), I am OK to leave it as it is.


https://reviews.llvm.org/D28637