From fe92b23328b0503ec4eb20a0d70a01dcf4fc0532 Mon Sep 17 00:00:00 2001
From: Andreas Kloeckner <inform@tiker.net>
Date: Tue, 1 Dec 2015 19:05:17 -0600
Subject: [PATCH] Leave a note about a possible error check

---
 loopy/kernel/data.py | 8 ++++++++
 1 file changed, 8 insertions(+)

diff --git a/loopy/kernel/data.py b/loopy/kernel/data.py
index 66f26a6d9..a777f4584 100644
--- a/loopy/kernel/data.py
+++ b/loopy/kernel/data.py
@@ -760,6 +760,14 @@ class ExpressionInstruction(InstructionBase):
         if isinstance(expression, str):
             assignee = parse(expression)
 
+        # FIXME: It may be worth it to enable this check eventually.
+        # For now, it causes grief with certain 'checky' uses of the
+        # with_transformed_expressions(). (notably the access checker)
+        #
+        # from pymbolic.primitives import Variable, Subscript
+        # if not isinstance(assignee, (Variable, Subscript)):
+        #     raise LoopyError("invalid lvalue '%s'" % assignee)
+
         self.assignee = assignee
         self.expression = expression
         self.temp_var_type = temp_var_type
-- 
GitLab