Skip to content

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