NFC.
Details
Diff Detail
- Build Status
Buildable 14889 Build 14889: arc lint + arc unit
Event Timeline
Some comments, but mainly I'm just bewildered by tablegen.
llvm/lib/Target/NVPTX/NVPTXISelDAGToDAG.cpp | ||
---|---|---|
3636 | Is it worth having a comment on this one? | |
llvm/lib/Target/NVPTX/NVPTXIntrinsics.td | ||
7420–7460 | Can we make WithStride a bool, or is that not supported? | |
7426 | It's sort of a nit, but: I think I see what you're trying to do with the alignment of # under the paren, but it doesn't totally work for me, especially when we add a space *after* the #. Maybe you could just pull the #s up onto the line above? Then we don't have any of these issues. | |
7549 | Dead code? |
Updated to address Justin's comments.
llvm/lib/Target/NVPTX/NVPTXIntrinsics.td | ||
---|---|---|
7420–7460 | Alas, there are no bools in tablegen. I can replace it with 'bit', though. |
llvm/lib/Target/NVPTX/NVPTXIntrinsics.td | ||
---|---|---|
7437 | "that will match" or "that matches" |
Is it worth having a comment on this one?