Merge pull request #509 from facebookresearch/pr/clean_up

minor clean-ups

Author: ftynse

tc/core/compiler.h

@@ -32,7 +32,7 @@
  *   3. parse a TC definition and retrieve the map of TC function to parsed TC
  *      trees.
  *
- * Compilation is backed by a compilation cache, its correspondance is:
+ * Compilation is backed by a compilation cache, its correspondence is:
  * 1 TcExecutor <-> 1 compiled tuple<TC function, input shapes, MappingOptions>
  *
  * The compile function is templated by the Backend type.

tc/core/halide2isl.h

@@ -93,7 +93,7 @@ ScheduleTreeAndAccesses makeScheduleTree(
     const Halide::Internal::Stmt& s);
 
 /// Enumerate all reductions in a statement, by looking for the
-/// ReductionInit and ReductionUpdate markers inserted during lowering
+/// ReductionUpdate markers inserted during lowering
 /// (see tc2halide.h).
 /// Each reduction object stores a reference to
 /// the update statement, and a list of reduction dimensions

tc/core/polyhedral/memory_promotion.h

@@ -38,7 +38,7 @@ enum class AccessType : short { Read, Write };
 // access is not strided, then "offset" is a zero expression and "stride" is 1.
 // The lowerBound and the size are computed after removing the potential stride.
 // The scope is defined by a specific position in a schedule tree (const
-// ScheduleTree*), the user is responsible for maintaining the correspondance
+// ScheduleTree*), the user is responsible for maintaining the correspondence
 // between schedule tree positions and footprints.
 struct ScopedFootprint {
   size_t dim() const {
@@ -66,7 +66,7 @@ struct ScopedFootprint {
 
 // Descriptor of tensor reference in a Scop.
 // May be scoped to a specific position in a schedule tree, the user is
-// responsible for maintaining the correspondance between schedule tree
+// responsible for maintaining the correspondence between schedule tree
 // positions and scoped access relations.
 class TensorReference {
  public:
@@ -104,7 +104,7 @@ typedef std::unordered_map<isl::id, TensorGroupsInfo, isl::IslIdIslHash>
 // memory together to avoid inconsistent values.
 //
 // Scoped to a specific position in a schedule tree, the user is responsible
-// for maintaing the correspondance between schedule tree positions and scoped
+// for maintaining the correspondence between schedule tree positions and scoped
 // access relations of each reference as well as scoped footprints.
 class TensorReferenceGroup {
  private:

tc/core/polyhedral/schedule_tree_elem.h

@@ -260,7 +260,6 @@ struct ScheduleTreeElemBand : public ScheduleTreeElemBase {
   // Drop the "n" dimensions starting at "pos" from "band".
   // We apply the transformation even if "n" is zero to ensure consistent
   // behavior with respect to changes in the schedule space.
-  // The caller is responsible for updating the isolate option (Note: why?)
   void drop(size_t pos, size_t n);
 
  public:

tc/core/polyhedral/scop.cc

@@ -103,10 +103,12 @@ std::ostream& operator<<(std::ostream& os, const Scop& s) {
   for (auto i : s.halide.idx) {
     os << i << ", ";
   }
+  os << "}, ";
   os << "reductionIdx: { ";
   for (auto i : s.halide.reductionIdx) {
     os << i << ", ";
   }
+  os << "}, ";
   os << "params: {";
   for (auto p : s.halide.params) {
     os << p.type() << " " << p.name() << ", ";