[PATCH] D46838: [MachineScheduler] Clear subregister entries also in addPhysRegDeps when handling a def

Jonas Paulsson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 15 03:52:09 PDT 2018


jonpa added a comment.

In https://reviews.llvm.org/D46838#1098248, @atrick wrote:

> We didn't support cross-call scheduling last I looked at this. Hopefully @MatzeB can verify that your fix is correct.


I did not intend to change anything in regards to the handling of calls, just to also handle subregs with Uses and Defs. Did I miss something?

Is there some other user of ScheduleDAGInstrs::buildSchedGraph() than MachineScheduler that allows rescheduling around calls, and that's why the handling of calls is here?


https://reviews.llvm.org/D46838





More information about the llvm-commits mailing list