[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