-
Notifications
You must be signed in to change notification settings - Fork 10.4k
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
Drop references to clang-pseudo-gen after removal #77070
Drop references to clang-pseudo-gen after removal #77070
Conversation
pseudo was removed from clang-tools-extra in upstream PR llvm/llvm-project#109154 Drop two explicit references in swift and unblock Windows CI
@hjyamauchi @compnerd I think that will fix the Windows CI failures from these PRs: |
@bnbarham Last year you added the CMake option in |
@swift-ci please test |
I would have only added it to fix builds during rebranch, so if it was removed there shouldn't be any reason to keep it 👍. Though I'm confused this would fix anything as main is currently on stable/20230725. Rebranch is on stable/20240723 and that PR wouldn't be in there I believe. EDIT: Oh you referenced a |
@hjyamauchi All tests have passed and this is ready to land. Can you merge please? Thanks |
I'm still a little unsure of what you're trying to do here, but again, |
@bnbarham Interesting, thanks for your note. Is that the general approach? Or only for the time until the rebranch is merged? I would normally assume to go the other way around: land new patches on |
As I understand, there's an auto-merger from upstream llvm to the |
If the change can be made upstream then the usual approach would be to do that and then cherry-pick to the stable branches. But for Swift-specific LLDB changes (which need to also compile the swift compiler) it's the other way around (stable -> next), as swift's |
Going to revert this patch to unblock CI. https://ci.swift.org/job/oss-swift-package-macos/3546/changes#6437e32e2c6a9b1c7c1b423f7768e678574ea98f |
Well, this bot is building the same swift against llvm-project |
pseudo was removed from clang-tools-extra in upstream PR llvm/llvm-project#109154
Drop two explicit references in swift and unblock Windows CI