commit 6adaca60626f53b1388a99fd05db23ba350ea4ac
author Dave Rodgman <dave.rodgman@arm.com> Wed May 24 19:43:13 2023 +0100
committer Dave Rodgman <dave.rodgman@arm.com> Wed May 24 19:53:21 2023 +0100
tree dc3adc1a8b8767891b1c45b30500ae4c3b09cc2e
parent ffbb7c5edc779df135f477bcf0a4e700184aa850

Minor tidy-up

Signed-off-by: Dave Rodgman <dave.rodgman@arm.com>

library/bn_mul.h

diff --git a/library/bn_mul.h b/library/bn_mul.h
index e76a8fd..c6e9fab 100644
--- a/library/bn_mul.h
+++ b/library/bn_mul.h
@@ -670,11 +670,7 @@
  * x !=0, which we can detect using __OPTIMIZE__ (which is also defined by
  * clang and armcc5 under the same conditions).
  */
-
-//#if defined(__ARM_ARCH)
-//#if __ARM_ARCH >= 6
-
-#if defined(__thumb__) && !defined(__thumb2__) // Thumb1 (not Thumb 2) ISA
+#if defined(__thumb__) && !defined(__thumb2__) // Thumb 1 (not Thumb 2) ISA
 
     // Only supported by gcc, when optimisation is enabled; only option A works
     #if defined(__OPTIMIZE__) && !defined(__ARMCC_VERSION)
@@ -685,7 +681,7 @@
 
     #if !defined(__ARMCC_VERSION) && !defined(__OPTIMIZE__)
         // gcc -O0: only option B builds
-        #if defined (__ARM_FEATURE_DSP) && (__ARM_FEATURE_DSP == 1)
+        #if (__ARM_ARCH >= 6) && defined (__ARM_FEATURE_DSP) && (__ARM_FEATURE_DSP == 1)
         #define ARM_OPTION_B
         #endif
     #else
@@ -698,18 +694,16 @@
     // any option builds. A does not seem to work; B is about 2x faster than C (under emulation).
     #define ARM_OPTION_B_OR_C
 
-#endif
+#endif /* Arm ISA selection */
 
 #if defined(ARM_OPTION_B_OR_C)
+// Prefer B, if we have the right features for it
 #if (__ARM_ARCH >= 6) && defined (__ARM_FEATURE_DSP) && (__ARM_FEATURE_DSP == 1)
 #define ARM_OPTION_B
 #else
 #define ARM_OPTION_C
 #endif
-#endif
-
-//#endif
-//#endif
+#endif /* defined(ARM_OPTION_B_OR_C) */
 
 #if defined(ARM_OPTION_A)