[PATCH] D108372: [X86][SchedModels] Fix missing ReadAdvance for MULX and ADCX/ADOX (PR51494)

Roman Lebedev via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 20 08:30:02 PDT 2021


lebedev.ri accepted this revision.
lebedev.ri added a comment.
This revision is now accepted and ready to land.

In D108372#2957460 <https://reviews.llvm.org/D108372#2957460>, @andreadb wrote:

> In D108372#2957430 <https://reviews.llvm.org/D108372#2957430>, @lebedev.ri wrote:
>
>> Just checking: we don't have this problem with `SBB`?
>
> Sorry. I missed that first question.
>
> SBB was already fixed by this commit: https://reviews.llvm.org/rG7a1a35a1d1ae2e69769505c9f39910067c53d53b
> That was the commit that fixed PR51318 and PR51322.

Aha, i thought it was.

LG to me, thank you.

@RKSimon ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D108372



More information about the llvm-commits mailing list