Remove optimization around BN_rshift() to support ARM64 due to VS bug

bagajjal · bagajjal · commit 5e51ae308099 · 2018-03-27T12:21:52.000-07:00
diff --git a/crypto/bn/bn_shift.c b/crypto/bn/bn_shift.c
@@ -167,7 +167,9 @@ BN_lshift(BIGNUM *r, const BIGNUM *a, int n)
 	bn_check_top(r);
 	return (1);
 }
-
+#ifdef _M_ARM64
+#pragma optimize("", off)
+#endif
 int
 BN_rshift(BIGNUM *r, const BIGNUM *a, int n)
 {
@@ -216,3 +218,6 @@ BN_rshift(BIGNUM *r, const BIGNUM *a, int n)
 	bn_check_top(r);
 	return (1);
 }
+#ifdef _M_ARM64
+#pragma optimize("", on)
+#endif

Original file line number	Diff line number	Diff line change
`@@ -167,7 +167,9 @@ BN_lshift(BIGNUM r, const BIGNUM a, int n)`
`167`	`167`	`bn_check_top(r);`
`168`	`168`	`return (1);`
`169`	`169`	`}`
`170`		`-`
	`170`	`+#ifdef _M_ARM64`
	`171`	`+#pragma optimize("", off)`
	`172`	`+#endif`
`171`	`173`	`int`
`172`	`174`	`BN_rshift(BIGNUM r, const BIGNUM a, int n)`
`173`	`175`	`{`
`@@ -216,3 +218,6 @@ BN_rshift(BIGNUM r, const BIGNUM a, int n)`
`216`	`218`	`bn_check_top(r);`
`217`	`219`	`return (1);`
`218`	`220`	`}`
	`221`	`+#ifdef _M_ARM64`
	`222`	`+#pragma optimize("", on)`
	`223`	`+#endif`