On Monday, April 30, 2018 at 10:50:20 AM UTC-4, Luke Niwranski wrote:
>
> Hi Jeff,
>
> I followed your MSBuild steps and the test still failed for me (results 
> below).
>
> Try this to specify the platform toolset:
>
> 1. Open VS2017 x64 Developer Prompt
> 2. cd to cryptopp
> 3. msbuild 
> /p:Configuration=Release;Platform=x64,WindowsTargetPlatformVersion=10.0.16299.0,PlatformToolset=v141
>  
> cryptlib.vcxproj
> 4. msbuild 
> /p:Configuration=Release;Platform=x64,WindowsTargetPlatformVersion=10.0.16299.0,PlatformToolset=v141
>  
> cryptest.vcxproj
> 5. msbuild 
> /p:Configuration=Release;Platform=x64,WindowsTargetPlatformVersion=10.0.16299.0,PlatformToolset=v141
>  
> /t:CopyCryptestToRoot cryptest.vcxproj
>
> Note: depending on the Visual Studio updates, you may need to change your 
> WindowsTargetPlatformVersion to whichever version is available.  I don't 
> think this has any affect on how Crypt++ is built.
>
> Daniele, thanks for confirming this.  It's great to know I'm not the only 
> one experiencing this problem!
>
> I'm assuming the failure will be triggered on all machines running the 
> library, not just the machine used to build the solution, correct?  As 
> such, is this a problem that can be fixed?  Or is the solution to simply to 
> disable optimization and/or upgrade the older hardware?
>

OK, I can duplicate it using your msbuild commands and this patch:

$ git diff cpu.cpp
diff --git a/cpu.cpp b/cpu.cpp
index ef0ddf69..c5bc3601 100644
--- a/cpu.cpp
+++ b/cpu.cpp
@@ -242,7 +242,7 @@ void DetectX86Features()
        g_hasSSSE3 = g_hasSSE2 && ((cpuid1[2] & (1<< 9)) != 0);
        g_hasSSE41 = g_hasSSE2 && ((cpuid1[2] & (1<<19)) != 0);
        g_hasSSE42 = g_hasSSE2 && ((cpuid1[2] & (1<<20)) != 0);
-       g_hasAESNI = g_hasSSE2 && ((cpuid1[2] & (1<<25)) != 0);
+       //g_hasAESNI = g_hasSSE2 && ((cpuid1[2] & (1<<25)) != 0);^M
        g_hasCLMUL = g_hasSSE2 && ((cpuid1[2] & (1<< 1)) != 0);

        if (IsIntel(cpuid0))

I'll need to look at it in detail to see where things are going sideways.

Jeff 

-- 
You received this message because you are subscribed to "Crypto++ Users". More 
information about Crypto++ and this group is available at 
http://www.cryptopp.com and 
http://groups.google.com/forum/#!forum/cryptopp-users.
--- 
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 [email protected].
For more options, visit https://groups.google.com/d/optout.

Reply via email to