gh-Cyan4973/xxHash
Watch
1
0
Fork
You've already forked xxHash
 0
Code Issues 7 Pull requests 3 Releases 26 Wiki Activity

XXH3 doesn't compile with MSVC #179

New issue
Closed
opened 2019-03-16 04:51:51 -05:00 by mgieseki · 5 comments
mgieseki commented 2019-03-16 04:51:51 -05:00 (Migrated from github.com)

First of all thanks for adding XXH3. It's an awesome addition to xxHash. While the recent sources build fine with MinGW, they failed to compile with MSVC (version19.16.27027, Visual Studio 2017 15.9.9) due to some unsupported #pragma and __attribute__ constructs.
Here's the compiler output (sorry, I've only the German locale installed at the moment):

xxh3.h(148): error C2156: Pragma muss außerhalb der Funktion liegen
xxh3.h(616): error C2143: Syntaxfehler: Es fehlt ")" vor "("
xxh3.h(616): error C2059: Syntaxfehler: ")"
xxh3.h(617): error C2143: Syntaxfehler: Es fehlt ")" vor "Typ"
xxh3.h(617): error C2091: Funktionsergebnis ist eine Funktion
xxh3.h(618): error C2085: "XXH3_hashLong_64b": Nicht in der formalen Parameterliste enthalten
xxh3.h(618): error C2143: Syntaxfehler: Es fehlt ";" vor "{"
xxh3.h(619): error C2065: "seed": nichtdeklarierter Bezeichner
xxh3.h(621): error C2065: "data": nichtdeklarierter Bezeichner
xxh3.h(621): warning C4022: "XXH3_hashLong": Zeigertyp für übergebenen Parameter 2 stimmt nicht mit dem deklarierten Zeigertyp überein
xxh3.h(621): error C2065: "len": nichtdeklarierter Bezeichner
xxh3.h(625): error C2065: "len": nichtdeklarierter Bezeichner
xxh3.h(625): warning C4047: "return": Anzahl der Dereferenzierungen bei "int (__cdecl *)()" und "XXH64_hash_t" unterschiedlich
xxh3.h(741): error C2143: Syntaxfehler: Es fehlt ")" vor "("
xxh3.h(741): error C2059: Syntaxfehler: ")"
xxh3.h(742): error C2143: Syntaxfehler: Es fehlt ")" vor "Typ"
xxh3.h(742): error C2091: Funktionsergebnis ist eine Funktion
xxh3.h(742): error C2084: Funktion "int (__cdecl *__attribute__())()" hat bereits einen Funktionsrumpf
xxh3.h(616): note: Siehe vorherige Definition von "__attribute__"
xxh3.h(743): error C2085: "XXH3_hashLong_128b": Nicht in der formalen Parameterliste enthalten
xxh3.h(743): error C2143: Syntaxfehler: Es fehlt ";" vor "{"
xxh3.h(744): error C2065: "seed": nichtdeklarierter Bezeichner
xxh3.h(747): error C2065: "data": nichtdeklarierter Bezeichner
xxh3.h(747): warning C4022: "XXH3_hashLong": Zeigertyp für übergebenen Parameter 2 stimmt nicht mit dem deklarierten Zeigertyp überein
xxh3.h(747): error C2065: "len": nichtdeklarierter Bezeichner
xxh3.h(751): error C2065: "len": nichtdeklarierter Bezeichner
xxh3.h(752): error C2065: "len": nichtdeklarierter Bezeichner
xxh3.h(753): error C2440: "return": "XXH128_hash_t" kann nicht in "int (__cdecl *)()" konvertiert werden
xxh3.h(769): error C2440: "return": "int" kann nicht in "XXH128_hash_t" konvertiert werden
First of all thanks for adding XXH3. It's an awesome addition to xxHash. While the recent sources build fine with MinGW, they failed to compile with MSVC (version19.16.27027, Visual Studio 2017 15.9.9) due to some unsupported `#pragma` and `__attribute__` constructs. Here's the compiler output (sorry, I've only the German locale installed at the moment): ``` xxh3.h(148): error C2156: Pragma muss außerhalb der Funktion liegen xxh3.h(616): error C2143: Syntaxfehler: Es fehlt ")" vor "(" xxh3.h(616): error C2059: Syntaxfehler: ")" xxh3.h(617): error C2143: Syntaxfehler: Es fehlt ")" vor "Typ" xxh3.h(617): error C2091: Funktionsergebnis ist eine Funktion xxh3.h(618): error C2085: "XXH3_hashLong_64b": Nicht in der formalen Parameterliste enthalten xxh3.h(618): error C2143: Syntaxfehler: Es fehlt ";" vor "{" xxh3.h(619): error C2065: "seed": nichtdeklarierter Bezeichner xxh3.h(621): error C2065: "data": nichtdeklarierter Bezeichner xxh3.h(621): warning C4022: "XXH3_hashLong": Zeigertyp für übergebenen Parameter 2 stimmt nicht mit dem deklarierten Zeigertyp überein xxh3.h(621): error C2065: "len": nichtdeklarierter Bezeichner xxh3.h(625): error C2065: "len": nichtdeklarierter Bezeichner xxh3.h(625): warning C4047: "return": Anzahl der Dereferenzierungen bei "int (__cdecl *)()" und "XXH64_hash_t" unterschiedlich xxh3.h(741): error C2143: Syntaxfehler: Es fehlt ")" vor "(" xxh3.h(741): error C2059: Syntaxfehler: ")" xxh3.h(742): error C2143: Syntaxfehler: Es fehlt ")" vor "Typ" xxh3.h(742): error C2091: Funktionsergebnis ist eine Funktion xxh3.h(742): error C2084: Funktion "int (__cdecl *__attribute__())()" hat bereits einen Funktionsrumpf xxh3.h(616): note: Siehe vorherige Definition von "__attribute__" xxh3.h(743): error C2085: "XXH3_hashLong_128b": Nicht in der formalen Parameterliste enthalten xxh3.h(743): error C2143: Syntaxfehler: Es fehlt ";" vor "{" xxh3.h(744): error C2065: "seed": nichtdeklarierter Bezeichner xxh3.h(747): error C2065: "data": nichtdeklarierter Bezeichner xxh3.h(747): warning C4022: "XXH3_hashLong": Zeigertyp für übergebenen Parameter 2 stimmt nicht mit dem deklarierten Zeigertyp überein xxh3.h(747): error C2065: "len": nichtdeklarierter Bezeichner xxh3.h(751): error C2065: "len": nichtdeklarierter Bezeichner xxh3.h(752): error C2065: "len": nichtdeklarierter Bezeichner xxh3.h(753): error C2440: "return": "XXH128_hash_t" kann nicht in "int (__cdecl *)()" konvertiert werden xxh3.h(769): error C2440: "return": "int" kann nicht in "XXH128_hash_t" konvertiert werden ```
👍 1
ifduyue commented 2019-03-16 05:38:50 -05:00 (Migrated from github.com)

Same here https://ci.appveyor.com/project/duyue/python-xxhash/build/job/aruc62wqgfg4x5kn

Same here https://ci.appveyor.com/project/duyue/python-xxhash/build/job/aruc62wqgfg4x5kn
Cyan4973 commented 2019-03-16 07:04:31 -05:00 (Migrated from github.com)

Thanks for the report,
yes I'm aware of these issues with Visual,
and just built a Windows VM to debug them.

Thanks for the report, yes I'm aware of these issues with `Visual`, and just built a Windows VM to debug them.
👍 3
LambdAurora commented 2019-03-16 10:32:08 -05:00 (Migrated from github.com)

I have some build issues too with MSVC.

When trying to build with cmake and BUILD_SHARED_LIBS=OFF, I got errors and some with the dllimport I added a while ago. I think I will try to look what's causing the issue.

And here's some of the logs:

"D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj" (cible par défaut) (3) ->
(ClCompile cible) ->
  D:\workspace\c\xxHash\xxHash\xxhash.c(254): error C2491: 'XXH_versionNumber' : définition de fonction dllimport non autori
sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(431): error C2491: 'XXH32' : définition de fonction dllimport non autorisée [D:\work
space\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(455): error C2491: 'XXH32_createState' : définition de fonction dllimport non autori
sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(459): error C2491: 'XXH32_freeState' : définition de fonction dllimport non autorisé
e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(465): error C2491: 'XXH32_copyState' : définition de fonction dllimport non autorisé
e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(470): error C2491: 'XXH32_reset' : définition de fonction dllimport non autorisée [D
:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(485): error C2491: 'XXH32_update' : définition de fonction dllimport non autorisée [
D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(548): error C2491: 'XXH32_digest' : définition de fonction dllimport non autorisée [
D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(575): error C2491: 'XXH32_canonicalFromHash' : définition de fonction dllimport non
autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(582): error C2491: 'XXH32_hashFromCanonical' : définition de fonction dllimport non
autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(864): error C2491: 'XXH64' : définition de fonction dllimport non autorisée [D:\work
space\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(887): error C2491: 'XXH64_createState' : définition de fonction dllimport non autori
sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(891): error C2491: 'XXH64_freeState' : définition de fonction dllimport non autorisé
e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(897): error C2491: 'XXH64_copyState' : définition de fonction dllimport non autorisé
e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(902): error C2491: 'XXH64_reset' : définition de fonction dllimport non autorisée [D
:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(916): error C2491: 'XXH64_update' : définition de fonction dllimport non autorisée [
D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(976): error C2491: 'XXH64_digest' : définition de fonction dllimport non autorisée [
D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(1003): error C2491: 'XXH64_canonicalFromHash' : définition de fonction dllimport non
 autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  D:\workspace\c\xxHash\xxHash\xxhash.c(1010): error C2491: 'XXH64_hashFromCanonical' : définition de fonction dllimport non
 autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  d:\workspace\c\xxhash\xxhash\xxh3.h(635): error C2491: 'XXH3_64bits_withSeed' : définition de fonction dllimport non autor
isée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  d:\workspace\c\xxhash\xxhash\xxh3.h(669): error C2491: 'XXH3_64bits' : définition de fonction dllimport non autorisée [D:\
workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  d:\workspace\c\xxhash\xxhash\xxh3.h(769): error C2491: 'XXH3_128bits_withSeed' : définition de fonction dllimport non auto
risée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  d:\workspace\c\xxhash\xxhash\xxh3.h(807): error C2491: 'XXH3_128bits' : définition de fonction dllimport non autorisée [D:
\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]
  d:\workspace\c\xxhash\xxhash\xxh3.h(813): error C2491: 'XXH128' : définition de fonction dllimport non autorisée [D:\works
pace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]```
(And sorry, I only have the French locale installed.)
I have some build issues too with MSVC. When trying to build with cmake and `BUILD_SHARED_LIBS=OFF`, I got errors and some with the `dllimport` I added a while ago. I think I will try to look what's causing the issue. And here's some of the logs: ```"D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\ALL_BUILD.vcxproj" (cible par défaut) (1) -> "D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj" (cible par défaut) (3) -> (ClCompile cible) -> D:\workspace\c\xxHash\xxHash\xxhash.c(254): error C2491: 'XXH_versionNumber' : définition de fonction dllimport non autori sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(431): error C2491: 'XXH32' : définition de fonction dllimport non autorisée [D:\work space\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(455): error C2491: 'XXH32_createState' : définition de fonction dllimport non autori sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(459): error C2491: 'XXH32_freeState' : définition de fonction dllimport non autorisé e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(465): error C2491: 'XXH32_copyState' : définition de fonction dllimport non autorisé e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(470): error C2491: 'XXH32_reset' : définition de fonction dllimport non autorisée [D :\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(485): error C2491: 'XXH32_update' : définition de fonction dllimport non autorisée [ D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(548): error C2491: 'XXH32_digest' : définition de fonction dllimport non autorisée [ D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(575): error C2491: 'XXH32_canonicalFromHash' : définition de fonction dllimport non autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(582): error C2491: 'XXH32_hashFromCanonical' : définition de fonction dllimport non autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(864): error C2491: 'XXH64' : définition de fonction dllimport non autorisée [D:\work space\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(887): error C2491: 'XXH64_createState' : définition de fonction dllimport non autori sée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(891): error C2491: 'XXH64_freeState' : définition de fonction dllimport non autorisé e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(897): error C2491: 'XXH64_copyState' : définition de fonction dllimport non autorisé e [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(902): error C2491: 'XXH64_reset' : définition de fonction dllimport non autorisée [D :\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(916): error C2491: 'XXH64_update' : définition de fonction dllimport non autorisée [ D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(976): error C2491: 'XXH64_digest' : définition de fonction dllimport non autorisée [ D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(1003): error C2491: 'XXH64_canonicalFromHash' : définition de fonction dllimport non autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] D:\workspace\c\xxHash\xxHash\xxhash.c(1010): error C2491: 'XXH64_hashFromCanonical' : définition de fonction dllimport non autorisée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] d:\workspace\c\xxhash\xxhash\xxh3.h(635): error C2491: 'XXH3_64bits_withSeed' : définition de fonction dllimport non autor isée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] d:\workspace\c\xxhash\xxhash\xxh3.h(669): error C2491: 'XXH3_64bits' : définition de fonction dllimport non autorisée [D:\ workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] d:\workspace\c\xxhash\xxhash\xxh3.h(769): error C2491: 'XXH3_128bits_withSeed' : définition de fonction dllimport non auto risée [D:\workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] d:\workspace\c\xxhash\xxhash\xxh3.h(807): error C2491: 'XXH3_128bits' : définition de fonction dllimport non autorisée [D: \workspace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj] d:\workspace\c\xxhash\xxhash\xxh3.h(813): error C2491: 'XXH128' : définition de fonction dllimport non autorisée [D:\works pace\c\xxHash\xxHash\cmake_unofficial\build\xxhash.vcxproj]``` (And sorry, I only have the French locale installed.)
👍 1
LambdAurora commented 2019-03-16 11:00:38 -05:00 (Migrated from github.com)

Hello again, I made a fix for this issue: #181. Now it compiles fine with MSVC (Visual Studio 15 2017 v15.9.21).

Hello again, I made a fix for this issue: #181. Now it compiles fine with MSVC (Visual Studio 15 2017 v15.9.21).
👍 1
Cyan4973 commented 2019-03-16 12:07:52 -05:00 (Migrated from github.com)

Thanks @LambdAurora ,

compilation errors under Visual should be fixed by now.

Thanks @LambdAurora , compilation errors under Visual should be fixed by now.
👍 2
Sign in to join this conversation.
No Branch/Tag specified
Branches Tags
dev
dependabot/github_actions/actions/upload-artifact-7.0.2
dependabot/github_actions/actions/download-artifact-8.0.2
dependabot/github_actions/actions/setup-node-7.1.0
archive/20261011T062114Z-rescue/fhgffy/fix/cmake-dispatch-off
dependabot/github_actions/msys2/setup-msys2-2.33.0
archive/20261011T062114Z-rescue/filtede98/fix-null-guards
ci/release-windows
fix/mingw-exe-symlink
gh-pages
release
ci/pin-actions
docs/refresh-readme
archive/20261011T062114Z-recovered/docs/refresh-readme
v0.8.4
v0.8.3
v0.8.2
v0.8.1
v0.8.0
v0.7.4
v0.7.3
v0.7.2
v0.7.1
v0.7.0
graphs
v0.6.5
v0.6.4
v0.6.3
v0.6.2
v0.6.1
v0.6.0
v0.5.1
v0.5.0
r42
r41
r40
r39
r38
r37
r36
r35
Labels
Clear labels   
announcement

   
bug

   
build

   
CI
Continuous Integration tests

  
dependencies
Pull requests that update a dependency file

  
documentation

   
duplicate

   
enhancement

   
experience feedback

   
feature request

   
github_actions
Pull requests that update GitHub Actions code

  
help wanted

   
install

   
invalid

   
Long-term

   
optimization

   
portability

   
question

   
target-specific
issue triggered in a specific environment (harware, compiler, OS)

  
test issue

   
wontfix
No labels
announcement
bug
build
CI
dependencies
documentation
duplicate
enhancement
experience feedback
feature request
github_actions
help wanted
install
invalid
Long-term
optimization
portability
question
target-specific
test issue
wontfix
Milestone
Clear milestone
No items
No milestone
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
gh-Cyan4973/xxHash#179
No description provided.