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

Move target feature whitelist from cg_llvm to cg_ssa #77521

Merged
merged 1 commit into from
Oct 4, 2020

Conversation

bjorn3
Copy link
Member

@bjorn3 bjorn3 commented Oct 4, 2020

These target features have to be supported or at least emulated by alternative codegen backends anyway as they are used by common crates. By moving this list to cg_ssa, other codegen backends don't have to copy
this code.

@rust-highfive
Copy link
Collaborator

r? @lcnr

(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 Oct 4, 2020
@bjorn3 bjorn3 force-pushed the target_feature_whitelist branch from fc9c5f0 to ae480c8 Compare October 4, 2020 09:21
@lcnr
Copy link
Contributor

lcnr commented Oct 4, 2020

ci is failing, otherwise LGTM

@bjorn3 bjorn3 force-pushed the target_feature_whitelist branch from ae480c8 to 0eb01af Compare October 4, 2020 09:35
@bjorn3
Copy link
Member Author

bjorn3 commented Oct 4, 2020

I forgot that I had disabled the llvm backend while working on a PR to build cg_clif in tree.

These target features have to be supported or at least emulated by
alternative codegen backends anyway as they are used by common crates.
By moving this list to cg_ssa, other codegen backends don't have to copy
this code.
@bjorn3 bjorn3 force-pushed the target_feature_whitelist branch from 0eb01af to 17d1cbb Compare October 4, 2020 09:49
@lcnr
Copy link
Contributor

lcnr commented Oct 4, 2020

@bors r+ rollup

@bors
Copy link
Contributor

bors commented Oct 4, 2020

📌 Commit 17d1cbb has been approved by lcnr

@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 Oct 4, 2020
bors added a commit to rust-lang-ci/rust that referenced this pull request Oct 4, 2020
…as-schievink

Rollup of 8 pull requests

Successful merges:

 - rust-lang#77072 (Minor `hash_map` doc adjustments + item attribute orderings)
 - rust-lang#77368 (Backport LLVM apfloat commit to rustc_apfloat)
 - rust-lang#77445 (BTreeMap: complete the compile-time test_variance test case)
 - rust-lang#77504 (Support vectors with fewer than 8 elements for simd_select_bitmask)
 - rust-lang#77513 (Change DocFragments from enum variant fields to structs with a nested enum)
 - rust-lang#77518 (Only use Fira Sans for the first `td` in item lists)
 - rust-lang#77521 (Move target feature whitelist from cg_llvm to cg_ssa)
 - rust-lang#77525 (Enable RenameReturnPlace MIR optimization on mir-opt-level >= 2)

Failed merges:

r? `@ghost`
@bors bors merged commit 5889cf3 into rust-lang:master Oct 4, 2020
@rustbot rustbot added this to the 1.49.0 milestone Oct 4, 2020
@bjorn3 bjorn3 deleted the target_feature_whitelist branch October 4, 2020 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
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.

5 participants