Z80_WITH_PARITY_COMPUTATION and Z80_WITH_PRECOMPUTED_DAA.

Author: redcode

.github/workflows/build-and-test-library.yml

@@ -38,6 +38,8 @@ env:
   Z80_WITH_EXECUTE: YES
   Z80_WITH_FULL_IM0: YES
   Z80_WITH_IM0_RETX_NOTIFICATIONS: YES
+  Z80_WITH_PARITY_COMPUTATION: NO
+  Z80_WITH_PRECOMPUTED_DAA: NO
   Z80_WITH_Q: YES
   Z80_WITH_SPECIAL_RESET: YES
   Z80_WITH_UNOFFICIAL_RETI: NO
@@ -84,6 +86,8 @@ jobs:
         -DZ80_WITH_EXECUTE=${{ env.Z80_WITH_EXECUTE }}
         -DZ80_WITH_FULL_IM0=${{ env.Z80_WITH_FULL_IM0 }}
         -DZ80_WITH_IM0_RETX_NOTIFICATIONS=${{ env.Z80_WITH_IM0_RETX_NOTIFICATIONS }}
+        -DZ80_WITH_PARITY_COMPUTATION=${{ env.Z80_WITH_PARITY_COMPUTATION }}
+        -DZ80_WITH_PRECOMPUTED_DAA=${{ env.Z80_WITH_PRECOMPUTED_DAA }}
         -DZ80_WITH_Q=${{ env.Z80_WITH_Q }}
         -DZ80_WITH_SPECIAL_RESET=${{ env.Z80_WITH_SPECIAL_RESET }}
         -DZ80_WITH_UNOFFICIAL_RETI=${{ env.Z80_WITH_UNOFFICIAL_RETI }}

.github/workflows/step-tests.yml

@@ -35,6 +35,8 @@ env:
   Z80_WITH_EXECUTE: YES
   Z80_WITH_FULL_IM0: YES
   Z80_WITH_IM0_RETX_NOTIFICATIONS: YES
+  Z80_WITH_PARITY_COMPUTATION: NO
+  Z80_WITH_PRECOMPUTED_DAA: NO
   Z80_WITH_Q: YES
   Z80_WITH_SPECIAL_RESET: YES
   Z80_WITH_UNOFFICIAL_RETI: NO
@@ -77,6 +79,8 @@ jobs:
         -DZ80_WITH_EXECUTE=${{ env.Z80_WITH_EXECUTE }}
         -DZ80_WITH_FULL_IM0=${{ env.Z80_WITH_FULL_IM0 }}
         -DZ80_WITH_IM0_RETX_NOTIFICATIONS=${{ env.Z80_WITH_IM0_RETX_NOTIFICATIONS }}
+        -DZ80_WITH_PARITY_COMPUTATION=${{ env.Z80_WITH_PARITY_COMPUTATION }}
+        -DZ80_WITH_PRECOMPUTED_DAA=${{ env.Z80_WITH_PRECOMPUTED_DAA }}
         -DZ80_WITH_Q=${{ env.Z80_WITH_Q }}
         -DZ80_WITH_SPECIAL_RESET=${{ env.Z80_WITH_SPECIAL_RESET }}
         -DZ80_WITH_UNOFFICIAL_RETI=${{ env.Z80_WITH_UNOFFICIAL_RETI }}

CMakeLists.txt

@@ -123,14 +123,13 @@ NO)
 option(${PROJECT_NAME}_WITH_PARITY_COMPUTATION
 "${PROJECT_NAME}: \
 Enable actual parity calculation for the P/V flag instead of using a \
-table of precomputed values (this is for benchmarks, DO NOT ENABLE in \
-production builds)."
+table of precomputed values (NOT RECOMMENDED for production builds)."
 NO)
 
 option(${PROJECT_NAME}_WITH_PRECOMPUTED_DAA
 "${PROJECT_NAME}: \
-Use a table of precomputed values in the `daa` instruction (this is for \
-benchmarks, DO NOT ENABLE in production builds)."
+Use a table of precomputed values to emulate the `daa` instruction (NOT \
+RECOMMENDED for production builds)."
 NO)
 
 option(${PROJECT_NAME}_WITH_Q

README

@@ -194,6 +194,16 @@ library by predefining macros that enable optional features:
         executed during the interrupt mode 0 response.
         The default is `NO`.
 
+    -DZ80_WITH_PARITY_COMPUTATION=(YES|NO)
+        Enable actual parity calculation for the P/V flag instead of using a
+        table of precomputed values (NOT RECOMMENDED for production builds).
+        The default is `NO`.
+
+    -DZ80_WITH_PRECOMPUTED_DAA=(YES|NO)
+        Use a table of precomputed values to emulate the `daa` instruction (NOT
+        RECOMMENDED for production builds).
+        The default is `NO`.
+
     -DZ80_WITH_Q=(YES|NO)
         Build the implementation of Q.
         The default is `NO`.
@@ -313,6 +323,8 @@ have the same name as their CMake equivalents:
     #define Z80_WITH_EXECUTE
     #define Z80_WITH_FULL_IM0
     #define Z80_WITH_IM0_RETX_NOTIFICATIONS
+    #define Z80_WITH_PARITY_COMPUTATION
+    #define Z80_WITH_PRECOMPUTED_DAA
     #define Z80_WITH_Q
     #define Z80_WITH_SPECIAL_RESET
     #define Z80_WITH_UNOFFICIAL_RETI
@@ -343,4 +355,4 @@ with this library. If not, see <http://www.gnu.org/licenses/>.
 
 
 ________________________________________________________________________________
-Last update: 2024-11-18                                               README EOF
+Last update: 2024-12-30                                               README EOF

README.md

@@ -470,6 +470,14 @@ If in doubt, read the [CMake documentation](https://cmake.org/documentation/) fo
 	Enable optional notifications for any `reti` or `retn` instruction executed during the interrupt mode 0 response.  
 	The default is `NO`.
 
+* <span id="cmake_option_z80_with_parity_computation">**`-DZ80_WITH_PARITY_COMPUTATION=(YES|NO)`**</span>  
+	Enable actual parity calculation for the P/V flag instead of using a table of precomputed values (NOT RECOMMENDED for production builds).  
+	The default is `NO`.
+
+* <span id="cmake_option_z80_with_precomputed_daa">**`-DZ80_WITH_PRECOMPUTED_DAA=(YES|NO)`**</span>  
+	Use a table of precomputed values to emulate the `daa` instruction (NOT RECOMMENDED for production builds).  
+	The default is `NO`.
+
 * <span id="cmake_option_z80_with_q">**`-DZ80_WITH_Q=(YES|NO)`**</span>  
 	Build the implementation of [Q](https://worldofspectrum.org/forums/discussion/41704).  
 	The default is `NO`.
@@ -649,6 +657,8 @@ The optional features of the emulator mentioned in "[Installation from sources](
 * **<code>#define [Z80_WITH_EXECUTE](#cmake_option_z80_with_execute)</code>**
 * **<code>#define [Z80_WITH_FULL_IM0](#cmake_option_z80_with_full_im0)</code>**
 * **<code>#define [Z80_WITH_IM0_RETX_NOTIFICATIONS](#cmake_option_z80_with_im0_retx_notifications)</code>**
+* **<code>#define [Z80_WITH_PARITY_COMPUTATION](#cmake_option_z80_with_parity_computation)</code>**
+* **<code>#define [Z80_WITH_PRECOMPUTED_DAA](#cmake_option_z80_with_precomputed_daa)</code>**
 * **<code>#define [Z80_WITH_Q](#cmake_option_z80_with_q)</code>**
 * **<code>#define [Z80_WITH_SPECIAL_RESET](#cmake_option_z80_with_special_reset)</code>**
 * **<code>#define [Z80_WITH_UNOFFICIAL_RETI](#cmake_option_z80_with_unofficial_reti)</code>**

documentation/installation-from-sources.rst

@@ -166,6 +166,16 @@ The second group of package-specific options configures the source code of the l
    Enable optional notifications for any ``reti`` or ``retn`` instruction executed during the interrupt mode 0 response. |br| |nl|
    The default is ``NO``.
 
+.. option:: -DZ80_WITH_PARITY_COMPUTATION=(YES|NO)
+
+   Enable actual parity calculation for the P/V flag instead of using a table of precomputed values (NOT RECOMMENDED for production builds). |br| |nl|
+   The default is ``NO``.
+
+.. option:: -DZ80_WITH_PRECOMPUTED_DAA=(YES|NO)
+
+   Use a table of precomputed values to emulate the ``daa`` instruction (NOT RECOMMENDED for production builds). |br| |nl|
+   The default is ``NO``.
+
 .. option:: -DZ80_WITH_Q=(YES|NO)
 
    Build the implementation of `Q <https://worldofspectrum.org/forums/discussion/41704>`_. |br| |nl|

documentation/integration.rst

@@ -86,6 +86,14 @@ The :ref:`optional features <Introduction:Optional features>` of the emulator me
 
    Enables optional notifications for any ``reti`` or ``retn`` instruction executed during the interrupt mode 0 response.
 
+.. c:macro:: Z80_WITH_PARITY_COMPUTATION
+
+   Enables actual parity calculation for the P/V flag instead of using a table of precomputed values.
+
+.. c:macro:: Z80_WITH_PRECOMPUTED_DAA
+
+   Enables the use of a table of precomputed values to emulate the ``daa`` instruction.
+
 .. c:macro:: Z80_WITH_Q
 
    Enables the implementation of `Q <https://worldofspectrum.org/forums/discussion/41704>`_.