From: cuz <cuz@b7a2c559-68d2-44c3-8de9-860c34a00d81>
Date: Sun, 10 Aug 2003 16:12:04 +0000 (+0000)
Subject: Fixed a bug
X-Git-Tag: V2.12.0~1467
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=38e102d2df571dbcfe499f2a57b1228d0018804e;p=cc65

Fixed a bug


git-svn-id: svn://svn.cc65.org/cc65/trunk@2256 b7a2c559-68d2-44c3-8de9-860c34a00d81
---

diff --git a/src/cc65/coptstop.c b/src/cc65/coptstop.c
index 5a512cb20..4f83b8ef5 100644
--- a/src/cc65/coptstop.c
+++ b/src/cc65/coptstop.c
@@ -209,10 +209,12 @@ static void CheckDirectOp (StackOpData* D)
             /* These insns are all ok and replaceable */
             D->Flags |= OP_DIRECT;
         } else if (E->AM == AM65_ZP_INDY && RegValIsKnown (E->RI->In.RegY) &&
-                   strcmp (E->Arg, D->ZPLo) != 0 && strcmp (E->Arg, D->ZPHi) != 0) {
-            /* Load indirect with known offset is also ok, provided that
-             * the zeropage location used is not the same as the one we're
-             * using for the temp storage.
+                   strcmp (E->Arg, "sp") == 0) {
+            /* A load from the stack with known offset is also ok, but in this
+             * case we must reload the index register later. Please note that
+             * a load indirect via other zero page locations is not ok, since
+             * these locations may change between the push and the actual
+             * operation.
              */
             D->Flags |= (OP_DIRECT | OP_RELOAD_Y);
         }