Remove dead code from rustc_codegen_llvm and the LLVM wrapper#136653
Merged
bors merged 1 commit intorust-lang:masterfrom Feb 7, 2025
Merged
Remove dead code from rustc_codegen_llvm and the LLVM wrapper#136653bors merged 1 commit intorust-lang:masterfrom
bors merged 1 commit intorust-lang:masterfrom
Conversation
rustbot
added
S-waiting-on-review
Zalathar
reviewed
Feb 7, 2025
| pub struct OptimizationDiagnostic { | ||
| pub kind: OptimizationDiagnosticKind, | ||
| pub pass_name: String, | ||
| pub function: &'ll Value, |
Member
There was a problem hiding this comment.
This field should be kept, even if it's currently unused on the Rust side, because it reflects part of a larger LLVM data structure.
(Keeping C++ and Rust in sync is easier if we don't have ad-hoc gaps in individual structures and enums.)
Zalathar
reviewed
Feb 7, 2025
| Unsupported(&'ll DiagnosticInfo), | ||
|
|
||
| /// LLVM has other types that we do not wrap here. | ||
| UnknownDiagnostic(&'ll DiagnosticInfo), |
Member
There was a problem hiding this comment.
Similarly, this field shouldn't be removed, even if it's unused.
Zalathar
reviewed
Feb 7, 2025
Comment on lines
-77
to
-83
| pub enum LLVMMachineType { | ||
| AMD64 = 0x8664, | ||
| I386 = 0x14c, | ||
| ARM64 = 0xaa64, | ||
| ARM64EC = 0xa641, | ||
| ARM = 0x01c0, | ||
| } |
Member
There was a problem hiding this comment.
Historical note: It looks like we used to pass this to LLVM code for emitting import libraries on Windows, and it probably became unnecessary after #129164. Good to remove.
Zalathar
reviewed
Feb 7, 2025
Comment on lines
-651
to
-656
| pub enum TailCallKind { | ||
| None, | ||
| Tail, | ||
| MustTail, | ||
| NoTail, | ||
| } |
Member
There was a problem hiding this comment.
Remark: This removal was justified in #135502 (comment), so good to remove.
Zalathar
reviewed
Feb 7, 2025
| } | ||
|
|
||
| pub type DiagnosticHandlerTy = unsafe extern "C" fn(&DiagnosticInfo, *mut c_void); | ||
| pub type InlineAsmDiagHandlerTy = unsafe extern "C" fn(&SMDiagnostic, *const c_void, c_uint); |
Member
There was a problem hiding this comment.
Historical note: Looks like this became unused in #100460.
Zalathar
reviewed
Feb 7, 2025
|
|
||
| pub fn LLVMRustGetElementTypeArgIndex(CallSite: &Value) -> i32; | ||
|
|
||
| pub fn LLVMRustIsBitcode(ptr: *const u8, len: usize) -> bool; |
Zalathar
reviewed
Feb 7, 2025
| pub type DIFile = DIScope; | ||
| pub type DILexicalBlock = DIScope; | ||
| pub type DISubprogram = DIScope; | ||
| pub type DINameSpace = DIScope; |
Member
|
The changes to |
dpaoliello
force-pushed
the
cleanllvm1
branch
from
73d5478 to
2a6b274
Compare
Contributor
Author
Done |
Member
|
Thanks! @bors r+ rollup |
Collaborator
bors
added
S-waiting-on-bors
matthiaskrgr
added a commit
to matthiaskrgr/rust
that referenced
this pull request
Feb 7, 2025
Remove dead code from rustc_codegen_llvm and the LLVM wrapper First step to clean up the LLVM wrapper: remove existing dead code. Split out of rust-lang#135502 r? `@Zalathar`
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Feb 7, 2025
…iaskrgr Rollup of 8 pull requests Successful merges: - rust-lang#134367 (Stabilize `feature(trait_upcasting)`) - rust-lang#135354 ([Debuginfo] Add MSVC Synthetic and Summary providers to LLDB) - rust-lang#135940 (Update toolstate maintainers) - rust-lang#135945 (Remove some unnecessary parens in `assert!` conditions) - rust-lang#136577 (Pattern Migration 2024: try to suggest eliding redundant binding modifiers) - rust-lang#136598 (Fix suggestion for `dependency_on_unit_never_type_fallback` involving closures + format args expansions) - rust-lang#136653 (Remove dead code from rustc_codegen_llvm and the LLVM wrapper) - rust-lang#136664 (replace one `.map_or(true, ...)` with `.is_none_or(...)`) Failed merges: - rust-lang#136193 (Implement pattern type ffi checks) r? `@ghost` `@rustbot` modify labels: rollup
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Feb 7, 2025
…iaskrgr Rollup of 7 pull requests Successful merges: - rust-lang#134367 (Stabilize `feature(trait_upcasting)`) - rust-lang#135940 (Update toolstate maintainers) - rust-lang#135945 (Remove some unnecessary parens in `assert!` conditions) - rust-lang#136577 (Pattern Migration 2024: try to suggest eliding redundant binding modifiers) - rust-lang#136598 (Fix suggestion for `dependency_on_unit_never_type_fallback` involving closures + format args expansions) - rust-lang#136653 (Remove dead code from rustc_codegen_llvm and the LLVM wrapper) - rust-lang#136664 (replace one `.map_or(true, ...)` with `.is_none_or(...)`) r? `@ghost` `@rustbot` modify labels: rollup
rust-timer
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Feb 7, 2025
Rollup merge of rust-lang#136653 - dpaoliello:cleanllvm1, r=Zalathar Remove dead code from rustc_codegen_llvm and the LLVM wrapper First step to clean up the LLVM wrapper: remove existing dead code. Split out of rust-lang#135502 r? ``@Zalathar``
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
4 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
First step to clean up the LLVM wrapper: remove existing dead code.
Split out of #135502
r? @Zalathar