Set signext or zeroext for integer arguments on RISC-V and LoongArch64#131955
Merged
bors merged 3 commits intorust-lang:masterfrom Oct 24, 2024
Merged
Set signext or zeroext for integer arguments on RISC-V and LoongArch64#131955bors merged 3 commits intorust-lang:masterfrom
signext or zeroext for integer arguments on RISC-V and LoongArch64#131955bors merged 3 commits intorust-lang:masterfrom
Conversation
0ebbea6 to
3f98b30
Compare
Member
|
Oh thank you for 9ba8053 |
This comment has been minimized.
This comment has been minimized.
3f98b30 to
4bf70dd
Compare
Member
Author
|
CC @xen0n since the PR contains changes about LoongArch. |
signext or zeroext for integer arguments on RISC-Vsignext or zeroext for integer arguments on RISC-V and LoongArch64
nikic
reviewed
Oct 20, 2024
4bf70dd to
dfec22d
Compare
Collaborator
|
Some changes occurred in tests/codegen/sanitizer cc @rust-lang/project-exploit-mitigations, @rcvalle |
dfec22d to
4d7528f
Compare
nikic
reviewed
Oct 21, 2024
This comment has been minimized.
This comment has been minimized.
4d7528f to
6edf712
Compare
This comment has been minimized.
This comment has been minimized.
2354fad to
2dfd4d0
Compare
Member
|
@bors try |
Collaborator
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Oct 22, 2024
Set `signext` or `zeroext` for integer arguments on RISC-V and LoongArch64 Fixes rust-lang#114508. This PR contains 3 commits: - the first one introduces a new function `adjust_for_rust_abi` in `rustc_target`, and moves the x86 specific adjustment code into it; - the second one adds RISC-V specific adjustment code into it, which sets `signext` or `zeroext` attribute for integer arguments. - **UPDATE**: added the 3rd commit for apply the same adjustment for LoongArch64. r? `@workingjubilee` CC `@coastalwhite` `@Urgau` `@topperc` `@michaelmaitland` try-job: dist-loongarch64-linux try-job: dist-riscv64-linux try-job: test-various try-job: i686-gnu-nopt try-job: i686-gnu
Collaborator
|
☀️ Try build successful - checks-actions |
0adeaf3 to
f8cdf3c
Compare
Member
Author
|
Rebased to the latest master since #131871 is merged just now. |
f8cdf3c to
294a808
Compare
294a808 to
57bffe1
Compare
Member
So, looks like we're good. @bors r+ |
Collaborator
Collaborator
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Oct 23, 2024
…ingjubilee Set `signext` or `zeroext` for integer arguments on RISC-V and LoongArch64 This PR contains 3 commits: - the first one introduces a new function `adjust_for_rust_abi` in `rustc_target`, and moves the x86 specific adjustment code into it; - the second one adds RISC-V specific adjustment code into it, which sets `signext` or `zeroext` attribute for integer arguments. - **UPDATE**: added the 3rd commit to apply the same adjustment for LoongArch64.
Collaborator
|
The job Click to see the possible cause of the failure (guessed by this bot) |
Collaborator
|
💔 Test failed - checks-actions |
Member
|
@bors retry |
bors
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Oct 23, 2024
Rollup of 9 pull requests Successful merges: - rust-lang#130991 (Vectorized SliceContains) - rust-lang#131928 (rustdoc: Document `markdown` module.) - rust-lang#131955 (Set `signext` or `zeroext` for integer arguments on RISC-V and LoongArch64) - rust-lang#131979 (Minor tweaks to `compare_impl_item.rs`) - rust-lang#132036 (Add a test case for rust-lang#131164) - rust-lang#132039 (Specialize `read_exact` and `read_buf_exact` for `VecDeque`) - rust-lang#132060 ("innermost", "outermost", "leftmost", and "rightmost" don't need hyphens) - rust-lang#132065 (Clarify documentation of `ptr::dangling()` function) - rust-lang#132066 (Fix a typo in documentation of `pointer::sub_ptr()`) r? `@ghost` `@rustbot` modify labels: rollup
rust-timer
added a commit
to rust-lang-ci/rust
that referenced
this pull request
Oct 24, 2024
Rollup merge of rust-lang#131955 - SpriteOvO:riscv-int-arg-attr, r=workingjubilee Set `signext` or `zeroext` for integer arguments on RISC-V and LoongArch64 This PR contains 3 commits: - the first one introduces a new function `adjust_for_rust_abi` in `rustc_target`, and moves the x86 specific adjustment code into it; - the second one adds RISC-V specific adjustment code into it, which sets `signext` or `zeroext` attribute for integer arguments. - **UPDATE**: added the 3rd commit to apply the same adjustment for LoongArch64.
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
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.
This PR contains 3 commits:
adjust_for_rust_abiinrustc_target, and moves the x86 specific adjustment code into it;signextorzeroextattribute for integer arguments.Fixes #114508.
r? @workingjubilee
CC @coastalwhite @Urgau @topperc @michaelmaitland