[PATCH] D30350: [LSR] Add a cap for reassociation of AllFixupsOutsideLoop type LSRUse to protect compile time
Andrew Trick via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed Mar 8 13:48:45 PST 2017
atrick added a comment.
LSR is fundamentally combinatorial. It relies on arbitrary pruning. I don't like it, but that's the way it is, and we need to guard against pathological cases. I just wonder why you complete bypass the reassociation code instead of simply limiting the number of expressions that participate.
Repository:
rL LLVM
https://reviews.llvm.org/D30350
More information about the llvm-commits
mailing list