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

Apply [PATCH] [clang-repl] Add a interpreter-specific overload of op… #28

Closed

Conversation

Gnimuc
Copy link

@Gnimuc Gnimuc commented May 7, 2024

…erator new for C++.

@giordano
Copy link

I pushed 0007e48 which (a) retains authorship, (b) preserves full commit message, (c) references the upstream commit (I did a cherry-pick, instead of manually applying a patch).

@giordano giordano closed this May 25, 2024
@Gnimuc
Copy link
Author

Gnimuc commented May 25, 2024

Thanks! Could you also bump JLLs for LLVM 17 in Ygg?

@giordano
Copy link

giordano commented May 25, 2024

@Gnimuc I think we need also llvm#78843? Edit: maybe not, that's an internal test in llvm

@giordano
Copy link

Thanks! Could you also bump JLLs for LLVM 17 in Ygg?

That you can do yourself, I created the tag julia-17.0.6-5. take a look at JuliaPackaging/Yggdrasil#8697, JuliaPackaging/Yggdrasil#8705, JuliaPackaging/Yggdrasil#8715. The PRs have to be opened in that order, after the previous one was merged, and the corresponding PRs in General have been merged.

@Gnimuc
Copy link
Author

Gnimuc commented May 25, 2024

CppInterOp only needs this patch for llvm 17 at the moment.

@Gnimuc Gnimuc deleted the julia-release/17.x branch May 26, 2024 02:49
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