[PATCH] D102834: [SLP] Implement initial memory versioning.

Florian Hahn via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 27 08:12:08 PST 2022


fhahn added a comment.

In D102834#3258564 <https://reviews.llvm.org/D102834#3258564>, @SjoerdMeijer wrote:

> Hey, it's me again. :) 
> Any plans to pick this up, shall we get this in?

Hi, I am still planning to push this in, but I don't think it should happen before the branch for the next release which will happen soon.

I also would like to wrap up D109368 <https://reviews.llvm.org/D109368> first, which applies one of the fundamental ideas of this patch (optimistically generated RT checks, undo if not profitable) to LV and has in fact been going on for longer. There were a couple of inefficiencies in some of the generated runtime checks, which meant we did not estimate their cost accurately. Those should all be fixed now, and I hope D109368 <https://reviews.llvm.org/D109368> can go in soon after the branch. Once it has been in tree for a couple of weeks, we should proceed with this patch here IMO.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D102834



More information about the llvm-commits mailing list