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

Sanjay Patel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 22 09:55:48 PDT 2018


spatel created this revision.
spatel added reviewers: atrick, craig.topper, RKSimon, andreadb.
Herald added subscribers: javed.absar, mcrosier, MatzeB.

AFAICT, the TargetSchedModel is always initialized using the TargetSubtargetInfo's MCSchedModel and TargetInstrInfo, so we don't need to extract those and pass 3 parameters to init().

The relationship of the target classes isn't entirely clear to me yet, so I'm posting this small potential clean-up to make sure I'm not misunderstanding this already.


https://reviews.llvm.org/D44789

Files:
  include/llvm/CodeGen/TargetSchedule.h
  lib/CodeGen/IfConversion.cpp
  lib/CodeGen/MachineCombiner.cpp
  lib/CodeGen/MachineLICM.cpp
  lib/CodeGen/MachineTraceMetrics.cpp
  lib/CodeGen/ScheduleDAGInstrs.cpp
  lib/CodeGen/TargetSchedule.cpp
  lib/CodeGen/TargetSubtargetInfo.cpp
  lib/Target/AArch64/AArch64SIMDInstrOpt.cpp
  lib/Target/AArch64/AArch64StorePairSuppress.cpp
  lib/Target/SystemZ/SystemZMachineScheduler.cpp
  lib/Target/X86/X86CmovConversion.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D44789.139459.patch
Type: text/x-patch
Size: 6785 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180322/6ee69e72/attachment-0001.bin>


More information about the llvm-commits mailing list