[PATCH] D30011: [NVPTX] Unify vectorization of load/stores of aggregate arguments and return values.

Justin Lebar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Feb 18 11:29:50 PST 2017


jlebar accepted this revision.
jlebar added inline comments.
This revision is now accepted and ready to land.


================
Comment at: lib/Target/NVPTX/NVPTXISelLowering.cpp:190
+// load/store.
+// The flattened parameter is represented as a list of EVTs and
+// offsets, and the whole structure is aligned to ParamAlignment. This
----------------
Nit, please add blank line to separate paragraphs.


================
Comment at: lib/Target/NVPTX/NVPTXISelLowering.cpp:253
+// flattened function parameter or return value.
+// The flattened parameter is represented as the list of ValueVTs and
+// Offsets, and is aligned to ParamAlignment bytes. We return a vector
----------------
Newline to separate para


https://reviews.llvm.org/D30011





More information about the llvm-commits mailing list