[PATCH] D132750: [SLP]Fix PR57322: vectorize constant float stores.

Alexey Bataev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 29 11:31:21 PDT 2022


ABataev added a comment.

In D132750#3756195 <https://reviews.llvm.org/D132750#3756195>, @reames wrote:

> Adding this only for the store case seems undesirable both long term as it's clearly not the right long term goal and short term as it adds complexity for no good reason.  I'm not actively requesting you don't land this, but I do strongly want to encourage you to either a) do the analysis for the non-stores and refresh this patch, or b) commit to doing a follow up in the near future.

I can try to add OK_FPConstantValue value instead and change analysis using this value for now.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D132750



More information about the llvm-commits mailing list