[PATCH] D125712: [SLP][X86] Improve reordering to consider alternate instruction bundles
Simon Pilgrim via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu May 26 09:31:07 PDT 2022
RKSimon added a comment.
I'm concerned its yet something else that is being added to SLP to make up for a weakness some place else (usually the cost model mechanism) and may take years to unravel again.
In this case we don't have a way to determine realistic costs for instruction sequences - at best we can optionally provide operands when getting the cost of a single instruction - we already have this problem with folded loads, this could end up being a lot more complex :-(
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125712/new/
https://reviews.llvm.org/D125712
More information about the llvm-commits
mailing list