tree d760b76c3fbf91b0a7419dded267c941358cd6c3
parent 59b0fccb513aecab4dfb5ec4f0df7929997c274c
author Brian Smith <brian@briansmith.org> 1437942326 -0400
committer Adam Langley <agl@google.com> 1446848938 +0000

Remove non-ASM version of |bn_mul_mont| in bn/generic.c.

When building in OPENSSL_NO_ASM mode, MSVC complains about unreachable
code. The redundant initialization of |i| is the main problem. The
skipping of the first test of the condition |i < num| with |goto| was
also confusing.

It turns out that |bn_mul_mont| is only called when assembly language
optimizations are available, but in that case the assmebly language
versions will always be used instead. Although this code will be
compiled in |OPENSSL_NO_ASM| builds, it is never called in
|OPENSSL_NO_ASM| builds. Thus, it can just be removed.

Change-Id: Id551899b2602824978edc1a1cb0703b76516808d
Reviewed-on: https://boringssl-review.googlesource.com/5550
Reviewed-by: Adam Langley <agl@google.com>