From 2278ef90231c963b750924a30a28114ca6089ffc Mon Sep 17 00:00:00 2001
From: Kaushik Kulkarni <kaushikcfd@gmail.com>
Date: Wed, 8 Aug 2018 00:22:45 -0500
Subject: [PATCH] [ci skip] Added fixmes from yesterday's discussion.
---
loopy/program.py | 3 +++
loopy/statistics.py | 2 ++
2 files changed, 5 insertions(+)
diff --git a/loopy/program.py b/loopy/program.py
index e41d3830e..bb5b9b1ac 100644
--- a/loopy/program.py
+++ b/loopy/program.py
@@ -349,6 +349,9 @@ def rename_resolved_functions_in_a_single_kernel(kernel,
# {{{ program callables info
class ProgramCallablesInfo(ImmutableRecord):
+ # FIXME: dont evalutate num_times_called, rahter compute it from the
+ # resolved_functions
+ # FIXME: make the edit callables thing a ContextManager.
def __init__(self, resolved_functions, num_times_callables_called=None,
history=None, is_being_edited=False,
num_times_hit_during_editing={},
diff --git a/loopy/statistics.py b/loopy/statistics.py
index 6a9744a06..74cd1bc71 100755
--- a/loopy/statistics.py
+++ b/loopy/statistics.py
@@ -65,6 +65,8 @@ __doc__ = """
# - The variable name, what if multiple kernels use the same name?
# - We should also add the cumulative effect on the arguments of callee kernels
# into the caller kernel.
+# FIXME: add an error that there is only one callable kernel. disable for
+# multiple callable kernels.
# {{{ GuardedPwQPolynomial
--
GitLab