[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