[PATCH] D44789: [TargetSchedule] shrink interface for init(); NFCI

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 4 11:36:17 PDT 2018


spatel added a comment.

In https://reviews.llvm.org/D44789#1056522, @RKSimon wrote:

> I agree that the MCSchedModel argument can go - not certain about TargetInstrInfo though.


For reference, I was digging around here while looking at:
https://bugs.llvm.org/show_bug.cgi?id=36550
...I'm not sure if we'd want/need to use the TargetSchedule wrapper to solve that one, but this looked a bit ugly. If it's irrelevant or nobody cares otherwise, I can abandon.


https://reviews.llvm.org/D44789





More information about the llvm-commits mailing list