[PATCH] D38633: [TableGen] Allow intrinsics to have up to 8 return values.

Artem Belevich via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 6 13:01:33 PDT 2017


tra added inline comments.


================
Comment at: llvm/lib/IR/Function.cpp:655
+  IIT_STRUCT7 = 39,
+  IIT_STRUCT8 = 40
 };
----------------
jlebar wrote:
> Is there a reason not to put these consecutive with struct5?
Caution mostly. If enum has explicitly assigned values, I assume that someone somewhere relies on them being exactly those values and it's prudent to extend the range rather than change existing order. 




https://reviews.llvm.org/D38633





More information about the llvm-commits mailing list