[PATCH] D46130: [NVPTX] Turn on Loop/SLP vectorization

[Benjamin Kramer via Phabricator via llvm-commits]

bkramer added inline comments.


================
Comment at: lib/Target/NVPTX/NVPTXTargetTransformInfo.h:53
+  // NVPTX has infinite registers of all kinds.
+  unsigned getNumberOfRegisters(bool Vector) const { return 1; }
+
----------------
jlebar wrote:
> Does 1 have specific meaning?  I don't see this in any of the comments, and that would be a pretty weird API...  (Like, did you mean -1?)
1 is the default of the generic implementation, I just copied that and removed the check for vector. I'm not even sure if anyone ever checks the value or just compares it against zero.


Repository:
  rL LLVM

https://reviews.llvm.org/D46130