[PATCH] D124616: [TTI][X86] Fix splat-load cost when load+broadcast cannot be combined.
Vasileios Porpodas via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon May 2 08:23:52 PDT 2022
vporpo added a comment.
Yes, I think it makes sense to add the `hasOneUse` check.
I am not really sure about the BBs though, it was reported as a potential issue in SLP, but it turned out that it was a different issue after all. So I guess I will drop this check and keep.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124616/new/
https://reviews.llvm.org/D124616
More information about the llvm-commits
mailing list