[PATCH] D62633: Recommit r360171: [DAGCombiner] Avoid creating large tokenfactors in visitTokenFactor.
Nirav Dave via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Wed May 29 16:18:18 PDT 2019
niravd added a comment.
Can you get away with just adding all remaining TFs (not their operands) to Ops? That way we keep the TF smaller and the pruning search still happens. That seems sufficient to fix the dropped operator issue.
If my intuition is wrong and we do need to CompletelyExpanded you should disable the pruning search itself when it's not being used.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the llvm-commits