[PATCH] D71717: [MachineScheduler] Ignore artificial edges when forming store chains

Stanislav Mekhanoshin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Dec 19 16:59:22 PST 2019


rampitec added a comment.

This might be dangerous. You may cluster nodes in a way it will be impossible to schedule them according to the artificial edges. Then again, these edges are there for a reason, aren't they? I also do not understand why would it only affect stores, there does not seem to be a check.

Maybe logic of artificial edges creation needs to be revised instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71717





More information about the llvm-commits mailing list