[PATCH] D52886: [X86] Move ReadAfterLd functionality into X86FoldableSchedWrite (PR36957)

Andrea Di Biagio via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 4 08:06:12 PDT 2018


andreadb added a comment.

Thanks Simon!

I am happy to see how this fixes the issue with read-advance being set to 3cy instead of 5cy for vector loads in BtVer2.

I also like the overall design (i.e. how the SchedRead is set).
I let Clement and Craig comment on the changes to the Intel models.
>From my point of view, this patch looks good.

-Andrea


Repository:
  rL LLVM

https://reviews.llvm.org/D52886





More information about the llvm-commits mailing list