[PATCH] D56739: [SelectionDAG] Add getTokenFactor, which splits nodes with > 64k operands.

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jan 16 14:07:42 PST 2019


RKSimon added a comment.

In D56739#1360490 <https://reviews.llvm.org/D56739#1360490>, @fhahn wrote:

> In D56739#1359828 <https://reviews.llvm.org/D56739#1359828>, @RKSimon wrote:
>
> > @fhahn Are you intending to look at the similar SDNode::NumValues limit? PR7250 + PR37000 ?
>
>
> I am primarily trying to fix SelectionDAG to deal with programs with a very large number of stores at the moment. But I could have a look at the mentioned issues in a bit and see if the same solution is also applicable.


No problem, just trying to ensure those bugs stay on the radar. Thanks.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D56739/new/

https://reviews.llvm.org/D56739





More information about the llvm-commits mailing list