[llvm] r233744 - Clarify that higher priority means higher values.

Matthias Braun matze at braunis.de
Tue Mar 31 13:04:46 PDT 2015


Author: matze
Date: Tue Mar 31 15:04:46 2015
New Revision: 233744

URL: http://llvm.org/viewvc/llvm-project?rev=233744&view=rev
Log:
Clarify that higher priority means higher values.

Modified:
    llvm/trunk/include/llvm/Target/Target.td
    llvm/trunk/include/llvm/Target/TargetRegisterInfo.h

Modified: llvm/trunk/include/llvm/Target/Target.td
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/Target.td?rev=233744&r1=233743&r2=233744&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Target/Target.td (original)
+++ llvm/trunk/include/llvm/Target/Target.td Tue Mar 31 15:04:46 2015
@@ -209,9 +209,9 @@ class RegisterClass<string namespace, li
   code AltOrderSelect = [{}];
 
   // Specify allocation priority for register allocators using a greedy
-  // heuristic. Classes with high priority are assigned first. It is sometimes
-  // beneficial to assign registers to highly constrained classes first.
-  // The priority has to be in the range [0,63].
+  // heuristic. Classes with higher priority values are assigned first. This is
+  // useful as it is sometimes beneficial to assign registers to highly
+  // constrained classes first. The value has to be in the range [0,63].
   int AllocationPriority = 0;
 }
 

Modified: llvm/trunk/include/llvm/Target/TargetRegisterInfo.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/Target/TargetRegisterInfo.h?rev=233744&r1=233743&r2=233744&view=diff
==============================================================================
--- llvm/trunk/include/llvm/Target/TargetRegisterInfo.h (original)
+++ llvm/trunk/include/llvm/Target/TargetRegisterInfo.h Tue Mar 31 15:04:46 2015
@@ -46,8 +46,8 @@ public:
   const uint32_t *SubClassMask;
   const uint16_t *SuperRegIndices;
   const unsigned LaneMask;
-  /// Classes with high priority should be assigned first by register allocators
-  /// with a greedy heuristic. The priority is a value in the range [0,63].
+  /// Classes with a higher priority value are assigned first by register
+  /// allocators using a greedy heuristic. The value is in the range [0,63].
   const uint8_t AllocationPriority;
   /// Whether the class supports two (or more) disjunct subregister indices.
   const bool HasDisjunctSubRegs;





More information about the llvm-commits mailing list