[PATCH] D119965: [LICM][PhaseOrder] Don't speculate in LICM until after running loop rotate

Dávid Bolvanský via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 11:25:54 PDT 2022


xbolva00 added a comment.

In D119965#3408489 <https://reviews.llvm.org/D119965#3408489>, @Carrot wrote:

> @wsmoses, this patch caused eigen regression in our code, it seems some originally vectorized code changed to scalarized code.
> Could you revert it?

This patch improved many regressions so it would be bad trade off to rush things and revert it.

It would be more productive approach if you could prepare small repro or at least some steps to reproduce your issue.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119965



More information about the llvm-commits mailing list