[PATCH] D153117: [MISched] Fix bug(s) in bottom-up scheduling.

[Francesco Petrogalli via Phabricator via llvm-commits]

fpetrogalli added a subscriber: dmgreen.
fpetrogalli added a comment.

In D153117#4453678 <https://reviews.llvm.org/D153117#4453678>, @andreadb wrote:

> I cannot really comment on the codegen changes as I don't consider myself an expert of that particular target.
> Due to the nature of this patch, I am not surprised to see so many changes.
> That being said, your changes in MachineScheduler are correct. Patch look good to me.

Thank you @andreadb !

@dmgreen - do you have any concerns about the changes in the assembly below?

In D153117#4453678 <https://reviews.llvm.org/D153117#4453678>, @andreadb wrote:

> I cannot really comment on the codegen changes as I don't consider myself an expert of that particular target.
> Due to the nature of this patch, I am not surprised to see so many changes.
> That being said, your changes in MachineScheduler are correct. Patch look good to me.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153117