Skip to content

enable --disable-dependency-tracking on CYGWIN also #490

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

Merged
merged 1 commit into from
Apr 20, 2025

Conversation

jeremyd2019
Copy link
Contributor

it also takes an inordinate amount of time there.

@jeremyd2019
Copy link
Contributor Author

As you may have noticed by my pull requests on llvm/llvm-project, I've been playing with clang on Cygwin. I figured it'd be good exercise for it to run ./build-all.sh --host-clang and see what broke. Nothing did, but the dependency tracking stuff took a long time, as it does on MSYS2.

it also takes an inordinate amount of time there.
@mstorsjo
Copy link
Owner

LGTM, thanks! (I amended your patch to spell out your name properly, as it only had the username, not the full name, in the commit.) I'll merge it after letting the CI run for a while (even though I don't foresee it breaking).

@jeremyd2019
Copy link
Contributor Author

BTW, I was eventually able to use the --host-clang toolchain to then build-cross-tools for i686, but:

  1. I gave up on --with-python after several rounds fighting it - upstream cpython doesn't build out-of-the box on Cygwin, and then things went sideways trying to build cpython-mingw, not finding the python import lib.
  2. I had to include the Signals patch from [LLVM][Clang][Cygwin] Fix building Clang for Cygwin llvm/llvm-project#134494, and add cmake option -DCROSS_TOOLCHAIN_FLAGS_NATIVE="-DCMAKE_CXX_FLAGS=-D_GNU_SOURCE=1;-DCMAKE_C_FLAGS=-D_GNU_SOURCE=1" so the "NATIVE" tools (llvm-min-tblgen etc) would build.

@mstorsjo mstorsjo merged commit 0aaef5d into mstorsjo:master Apr 20, 2025
19 checks passed
@jeremyd2019 jeremyd2019 deleted the patch-1 branch April 20, 2025 20:31
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.

2 participants