[llvm] r332599 - [SystemZ] Commenting (NFC)

Jonas Paulsson via llvm-commits llvm-commits at lists.llvm.org
Thu May 17 04:53:56 PDT 2018


Author: jonpa
Date: Thu May 17 04:53:56 2018
New Revision: 332599

URL: http://llvm.org/viewvc/llvm-project?rev=332599&view=rev
Log:
[SystemZ]  Commenting (NFC)

Some minor commenting in scheduler files.

Review: Ulrich Weigand

Modified:
    llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ13.td
    llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ14.td

Modified: llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ13.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ13.td?rev=332599&r1=332598&r2=332599&view=diff
==============================================================================
--- llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ13.td (original)
+++ llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ13.td Thu May 17 04:53:56 2018
@@ -10,6 +10,8 @@
 // This file defines the machine model for Z13 to support instruction
 // scheduling and other instruction cost heuristics.
 //
+// Pseudos expanded right after isel do not need to be modelled here.
+//
 //===----------------------------------------------------------------------===//
 
 def Z13Model : SchedMachineModel {
@@ -28,9 +30,9 @@ def Z13Model : SchedMachineModel {
 
 let SchedModel = Z13Model in  {
 
-// These definitions could be put in a subtarget common include file,
-// but it seems the include system in Tablegen currently rejects
-// multiple includes of same file.
+// These definitions need the SchedModel value. They could be put in a
+// subtarget common include file, but it seems the include system in Tablegen
+// currently (2016) rejects multiple includes of same file.
 def : WriteRes<GroupAlone, []> {
   let NumMicroOps = 0;
   let BeginGroup  = 1;

Modified: llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ14.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ14.td?rev=332599&r1=332598&r2=332599&view=diff
==============================================================================
--- llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ14.td (original)
+++ llvm/trunk/lib/Target/SystemZ/SystemZScheduleZ14.td Thu May 17 04:53:56 2018
@@ -10,6 +10,8 @@
 // This file defines the machine model for Z14 to support instruction
 // scheduling and other instruction cost heuristics.
 //
+// Pseudos expanded right after isel do not need to be modelled here.
+//
 //===----------------------------------------------------------------------===//
 
 def Z14Model : SchedMachineModel {
@@ -28,9 +30,9 @@ def Z14Model : SchedMachineModel {
 
 let SchedModel = Z14Model in  {
 
-// These definitions could be put in a subtarget common include file,
-// but it seems the include system in Tablegen currently rejects
-// multiple includes of same file.
+// These definitions need the SchedModel value. They could be put in a
+// subtarget common include file, but it seems the include system in Tablegen
+// currently (2016) rejects multiple includes of same file.
 def : WriteRes<GroupAlone, []> {
   let NumMicroOps = 0;
   let BeginGroup  = 1;




More information about the llvm-commits mailing list