[PATCH] D155900: [TTI][NFCI] Introduce two new target transform hooks

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:815
+
+    // TODO: Is scalable vs non-scalable OK?
+    // TODO: Is sized vs non-sized OK?
----------------
I'd assume no 


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:816
+    // TODO: Is scalable vs non-scalable OK?
+    // TODO: Is sized vs non-sized OK?
+    // TODO: For sized and non-scalable, we could check the sizes.
----------------
I'd assume no. Do we even allow unsized parameters?


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

https://reviews.llvm.org/D155900