[PATCH] D80885: [VectorCombine] scalarizeBinop - support an all-constant src vector operand

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 4 07:41:53 PDT 2020


RKSimon added a comment.

In D80885#2073312 <https://reviews.llvm.org/D80885#2073312>, @lebedev.ri wrote:

> I'm having a hard time following `scalarizeBinop()` now.
>  Perhaps it can be refactored somehow?


I'm hesitant to start refactoring yet as an imminent change will be to add cmp support (PR37463) will require further changes that aren't clear yet. @spatel - any suggestions?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80885





More information about the llvm-commits mailing list