[PATCH] D16306: [TTI] Add getCacheLineSize

[Adam Nemet via llvm-commits]

anemet added inline comments.

================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:267
@@ -266,1 +266,3 @@
 
+  unsigned getCacheLineSize() { return 64; }
+
----------------
hulx2000 wrote:
> Return 64 by default is not good, it would be better if assert or return unsupported message by default, and only return 64 for PPC.
> 
> 
I don't see a precedent for doing such invasive things in the TTI layer itself.

I could return 0 (invalid).  That way clients can do the assert or disable any action based on this parameter.




http://reviews.llvm.org/D16306