[PATCH] D56738: [SelectionDAG] Update check in createOperands to reflect max() is a valid value.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 15 12:10:06 PST 2019


fhahn added a comment.

In D56738#1358379 <https://reviews.llvm.org/D56738#1358379>, @lebedev.ri wrote:

> Can any test case be added here, or would it be just too big to realistically test?


I have a file triggering the assert, but I think the cases where it is triggered require 64k load/stores in a single BB. But maybe there is another way to construct a node with more than 64k operands?


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

https://reviews.llvm.org/D56738





More information about the llvm-commits mailing list