[PATCH] D116688: [SLP]Excluded external uses from the reprdering estimation.

[Alexey Bataev via Phabricator via llvm-commits]

ABataev added a comment.

In D116688#3226339 <https://reviews.llvm.org/D116688#3226339>, @vporpo wrote:

> Yes, I agree, we need a better way of dealing with uses. We should consider the score first and only check for uses if the scores are exactly the same.
> Checking for all-vectorized uses versus not-all-vectorized also makes sense.
>
> Could you please add a TODO  (perhaps near line 1336?) with a brief description on how we should be dealing with uses?

I was going to add the analysis of the usage to the patch, have a quick prototype but it has some regressions, will work on the improvements tomorrow.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D116688/new/

https://reviews.llvm.org/D116688