[PATCH] D32594: Fix typo and place comment close to its target

Wei-Ren Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 27 06:56:32 PDT 2017


chenwj created this revision.
Herald added a subscriber: MatzeB.

Should be "per se" not "per say". Also place comment about ConvergingVLIWScheduler upon the class itself.


https://reviews.llvm.org/D32594

Files:
  lib/Target/Hexagon/HexagonMachineScheduler.h


Index: lib/Target/Hexagon/HexagonMachineScheduler.h
===================================================================
--- lib/Target/Hexagon/HexagonMachineScheduler.h
+++ lib/Target/Hexagon/HexagonMachineScheduler.h
@@ -32,14 +32,10 @@
 using namespace llvm;
 
 namespace llvm {
-//===----------------------------------------------------------------------===//
-// ConvergingVLIWScheduler - Implementation of the standard
-// MachineSchedStrategy.
-//===----------------------------------------------------------------------===//
 
 class VLIWResourceModel {
   /// ResourcesModel - Represents VLIW state.
-  /// Not limited to VLIW targets per say, but assumes
+  /// Not limited to VLIW targets per se, but assumes
   /// definition of DFA by a target.
   DFAPacketizer *ResourcesModel;
 
@@ -110,6 +106,11 @@
   void schedule() override;
 };
 
+//===----------------------------------------------------------------------===//
+// ConvergingVLIWScheduler - Implementation of the standard
+// MachineSchedStrategy.
+//===----------------------------------------------------------------------===//
+
 /// ConvergingVLIWScheduler shrinks the unscheduled zone using heuristics
 /// to balance the schedule.
 class ConvergingVLIWScheduler : public MachineSchedStrategy {


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32594.96915.patch
Type: text/x-patch
Size: 1270 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170427/ac965b33/attachment.bin>


More information about the llvm-commits mailing list