> I noticed the Makefile has a line (
> http://github.com/weidai11/cryptopp/blob/master/GNUmakefile#L146):
>
> ifneq ($(IS_SUN),0) 
> CXXFLAGS += -Wa,--divide # allow use of "/" operator 
> endif
>
> That is a GNU Assembler option. It seems that block has not been executed 
> in some time, including 5.6.2 and earlier:
>
>     $ git checkout CRYPTOPP_5_6_2
>     $ make -j 2
>     g++ -DNDEBUG -g -O2 -pipe -c cryptlib
>     ...
>
> My question is, should this be fixed?
>

Let me be the first to answer that :)

Yes, its needed on Solaris (and maybe others):

$ g++ -DNDEBUG -g2 -O2 -fPIC -Wa,--divide -pipe -c integer.cpp
$

And without:

$ g++ -DNDEBUG -g2 -O2 -fPIC -pipe -c integer.cpp
{standard input}: Assembler messages:
{standard input}:6405: Error: unbalanced brackets in operand 2.
{standard input}:6406: Error: unbalanced brackets in operand 2.
{standard input}:6407: Error: unbalanced brackets in operand 2.
{standard input}:6408: Error: unbalanced brackets in operand 2.
...

And those {standard input} lines:

$ cat -n <S file> | grep "6405"
  6405  LMul16:movdqa xmm1, [esi+16/2*(1-(16-2*(16/2))*2)*16];

Jeff

-- 
-- 
You received this message because you are subscribed to the "Crypto++ Users" 
Google Group.
To unsubscribe, send an email to cryptopp-users-unsubscr...@googlegroups.com.
More information about Crypto++ and this group is available at 
http://www.cryptopp.com.
--- 
You received this message because you are subscribed to the Google Groups 
"Crypto++ Users" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to cryptopp-users+unsubscr...@googlegroups.com.
For more options, visit https://groups.google.com/d/optout.

Reply via email to