Skip to content

add two new build flags to build clang and enable llvm plugins #87297

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
Aug 2, 2021

Conversation

ZuseZ4
Copy link
Member

@ZuseZ4 ZuseZ4 commented Jul 20, 2021

Based on the discussion here: https://rust-lang.zulipchat.com/#narrow/stream/131828-t-compiler/topic/Add.20configure.20flag.20to.20build.20clang/near/246439138

It allows building clang (which already is part of the llvm-project) based on the same llvm version which we use to build rustc.
It also allows enabling llvm's plugin interface, which is required for https://enzyme.mit.edu/.

There is no further integration beside of this basic build support.

@rust-highfive
Copy link
Contributor

r? @Mark-Simulacrum

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Jul 20, 2021
@ZuseZ4 ZuseZ4 force-pushed the new_build_flags branch from 876710b to 8f6827f Compare July 31, 2021 21:15
@Mark-Simulacrum
Copy link
Member

@bors r+

@bors
Copy link
Collaborator

bors commented Aug 1, 2021

📌 Commit 8f6827f has been approved by Mark-Simulacrum

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 1, 2021
@bors
Copy link
Collaborator

bors commented Aug 2, 2021

⌛ Testing commit 8f6827f with merge 71c86470dda17b5c0a0f46341937ade4295d5787...

@bors
Copy link
Collaborator

bors commented Aug 2, 2021

💔 Test failed - checks-actions

@rust-log-analyzer
Copy link
Collaborator

A job failed! Check out the build log: (web) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@bors bors added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Aug 2, 2021
@ZuseZ4
Copy link
Member Author

ZuseZ4 commented Aug 2, 2021

https://github.com/rust-lang-ci/rust/runs/3217367001?check_suite_focus=true

Not really sure what's the issue here, since I can't (?) expand the 'install aws cli' section further. I also just used this change under Ubuntu, so not sure about Apple, but to me that looks like a random failure?

@jyn514
Copy link
Member

jyn514 commented Aug 2, 2021

@bors retry

@bors bors added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 2, 2021
@bors
Copy link
Collaborator

bors commented Aug 2, 2021

⌛ Testing commit 8f6827f with merge 76d247c...

@bors
Copy link
Collaborator

bors commented Aug 2, 2021

☀️ Test successful - checks-actions
Approved by: Mark-Simulacrum
Pushing 76d247c to master...

@bors bors added the merged-by-bors This PR was explicitly merged by bors. label Aug 2, 2021
@bors bors merged commit 76d247c into rust-lang:master Aug 2, 2021
@rustbot rustbot added this to the 1.56.0 milestone Aug 2, 2021
@ZuseZ4 ZuseZ4 deleted the new_build_flags branch August 2, 2021 10:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merged-by-bors This PR was explicitly merged by bors. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants