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

[Vasileios Porpodas via Phabricator via llvm-commits]

vporpo added a comment.

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?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116688