Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Don't link against libgcc_s on Windows with GHC 9.4 and later #429

Merged
merged 2 commits into from Apr 25, 2022

Conversation

bgamari
Copy link
Contributor

@bgamari bgamari commented Apr 25, 2022

As of GHC 9.4 we use libc+++ and Clang on Windows.

As of GHC 9.4 we use libc+++ and Clang on Windows.
@bgamari bgamari force-pushed the wip/windows-clang branch 2 times, most recently from d36ce2e to f74ec15 Compare April 25, 2022 17:04
@Bodigrim
Copy link
Contributor

@bgamari

  1. Please use the same GHC version in both impl(). Currently the first one is impl(ghc < 9.4) and another one is impl(ghc < 9.3).
  2. The proposed patch enables cpp-options: -DSIMDUTF_NO_THREADS and cxx-options: -fno-exceptions for Windows + GHC 9.4. Is it intended?

@bgamari
Copy link
Contributor Author

bgamari commented Apr 25, 2022

Please use the same GHC version in both impl(). Currently the first one is impl(ghc < 9.4) and another one is impl(ghc < 9.3).

Sigh, quite right. I'll fix.

The proposed patch enables cpp-options: -DSIMDUTF_NO_THREADS and cxx-options: -fno-exceptions for Windows + GHC 9.4. Is it intended?

Hmm, yes, I didn't read that comment closely enough. Indeed we likely shouldn't use these on Windows.

@Bodigrim
Copy link
Contributor

@bgamari thanks, please rebase.

@Bodigrim Bodigrim merged commit 89feee7 into haskell:master Apr 25, 2022
@Bodigrim
Copy link
Contributor

Nevermind, I can squash on my end. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants