From fdda2a2476b77ce6c033bc22db725208500e92aa Mon Sep 17 00:00:00 2001
From: Andreas Kloeckner <inform@tiker.net>
Date: Thu, 6 Jun 2013 10:53:18 -0400
Subject: [PATCH] PEP8 loopy.check.

---
 loopy/check.py | 33 ++++++++++++++-------------------
 1 file changed, 14 insertions(+), 19 deletions(-)

diff --git a/loopy/check.py b/loopy/check.py
index 64d21b425..5d7ea2147 100644
--- a/loopy/check.py
+++ b/loopy/check.py
@@ -23,15 +23,11 @@ THE SOFTWARE.
 """
 
 
-
-
 from islpy import dim_type
 import islpy as isl
 from loopy.symbolic import WalkMapper
 
 
-
-
 # {{{ sanity checks run during scheduling
 
 def check_for_unused_hw_axes_in_insns(kernel):
@@ -74,9 +70,6 @@ def check_for_unused_hw_axes_in_insns(kernel):
                         ",".join(str(i) for i in local_axes_used)))
 
 
-
-
-
 def check_for_double_use_of_hw_axes(kernel):
     from loopy.kernel.data import UniqueTag
 
@@ -93,8 +86,6 @@ def check_for_double_use_of_hw_axes(kernel):
                 insn_tag_keys.add(key)
 
 
-
-
 def check_for_inactive_iname_access(kernel):
     from loopy.symbolic import DependencyMapper
     depmap = DependencyMapper()
@@ -110,11 +101,10 @@ def check_for_inactive_iname_access(kernel):
                     % insn.id)
 
 
-
-
 class WriteRaceConditionError(RuntimeError):
     pass
 
+
 def check_for_write_races(kernel):
     from loopy.symbolic import DependencyMapper
     from loopy.kernel.data import ParallelTag, GroupIndexTag, LocalIndexTagBase
@@ -150,14 +140,14 @@ def check_for_write_races(kernel):
 
         elif assignee_name in kernel.temporary_variables:
             temp_var = kernel.temporary_variables[assignee_name]
-            if temp_var.is_local == True:
+            if temp_var.is_local is True:
                 raceable_parallel_insn_inames = set(
                         iname
                         for iname in kernel.insn_inames(insn)
                         if isinstance(iname_to_tag(iname), ParallelTag)
                         and not isinstance(iname_to_tag(iname), GroupIndexTag))
 
-            elif temp_var.is_local == False:
+            elif temp_var.is_local is False:
                 raceable_parallel_insn_inames = set(
                         iname
                         for iname in kernel.insn_inames(insn)
@@ -185,6 +175,7 @@ def check_for_write_races(kernel):
                     "is/are not referenced in the lhs index"
                     % (insn.id, ",".join(race_inames)))
 
+
 def check_for_orphaned_user_hardware_axes(kernel):
     from loopy.kernel.data import LocalIndexTag
     for axis in kernel.local_sizes:
@@ -198,6 +189,7 @@ def check_for_orphaned_user_hardware_axes(kernel):
             raise RuntimeError("user-requested local hardware axis %d "
                     "has no iname mapped to it" % axis)
 
+
 def check_for_data_dependent_parallel_bounds(kernel):
     from loopy.kernel.data import ParallelTag
 
@@ -293,6 +285,7 @@ def check_bounds(kernel):
         acm(insn.expression)
         acm(insn.assignee)
 
+
 def check_write_destinations(kernel):
     for insn in kernel.instructions:
         wvar = insn.get_assignee_var_name()
@@ -318,6 +311,7 @@ def check_write_destinations(kernel):
 
 # }}}
 
+
 def run_automatic_checks(kernel):
     try:
         check_for_orphaned_user_hardware_axes(kernel)
@@ -336,12 +330,13 @@ def run_automatic_checks(kernel):
         print 75*"="
         raise
 
+
 # {{{ sanity-check for implemented domains of each instruction
 
 def check_implemented_domains(kernel, implemented_domains, code=None):
     from islpy import dim_type
 
-    from islpy import align_spaces, align_two
+    from islpy import align_two
 
     for insn_id, idomains in implemented_domains.iteritems():
         insn = kernel.id_to_insn[insn_id]
@@ -352,7 +347,8 @@ def check_implemented_domains(kernel, implemented_domains, code=None):
         for idomain in idomains[1:]:
             insn_impl_domain = insn_impl_domain | idomain
         assumption_non_param = isl.BasicSet.from_params(kernel.assumptions)
-        assumptions, insn_impl_domain = align_two(assumption_non_param, insn_impl_domain)
+        assumptions, insn_impl_domain = align_two(
+                assumption_non_param, insn_impl_domain)
         insn_impl_domain = (
                 (insn_impl_domain & assumptions)
                 .project_out_except(kernel.insn_inames(insn), [dim_type.set]))
@@ -415,12 +411,13 @@ def check_implemented_domains(kernel, implemented_domains, code=None):
 
 # }}}
 
+
 # {{{ user-invoked checks
 
 def get_problems(kernel, parameters):
     """
-    :return: *(max_severity, list of (severity, msg))*, where *severity* ranges from 1-5.
-        '5' means 'will certainly not run'.
+    :return: *(max_severity, list of (severity, msg))*, where *severity*
+        ranges from 1-5.  '5' means 'will certainly not run'.
     """
     msgs = []
 
@@ -474,8 +471,6 @@ def get_problems(kernel, parameters):
     return max_severity, msgs
 
 
-
-
 def check_kernels(kernel_gen, parameters={}, kill_level_min=5,
         warn_level_min=1):
     for kernel in kernel_gen:
-- 
GitLab