[PATCH] D32888: TableGen: Add support of Intrinsics with multiple returns

Wenbo Sun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 9 23:26:14 PDT 2017


wenbos added inline comments.


================
Comment at: utils/TableGen/CodeGenDAGPatterns.cpp:2208
 
+  unsigned NumResults = GetNumNodeResults(Operator, CDP);
+
----------------
hfinkel wrote:
> Can you please add a comment explaining why GetNumNodeResults need to be called here (and not, say, below, where it is now).
Hi Hal, comments are added in latest patch. 


https://reviews.llvm.org/D32888





More information about the llvm-commits mailing list