Set signext or zeroext for integer arguments on RISC-V and LoongArch64#131955
Merged
Conversation
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 20, 2024 01:18
0ebbea6 to
3f98b30
Compare
Member
|
Oh thank you for 9ba8053 |
This comment has been minimized.
This comment has been minimized.
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 20, 2024 21:17
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
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 21, 2024 11:37
4bf70dd to
dfec22d
Compare
Collaborator
|
Some changes occurred in tests/codegen/sanitizer cc @rust-lang/project-exploit-mitigations, @rcvalle |
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 21, 2024 11:46
dfec22d to
4d7528f
Compare
nikic
reviewed
Oct 21, 2024
This comment has been minimized.
This comment has been minimized.
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 21, 2024 13:15
4d7528f to
6edf712
Compare
This comment has been minimized.
This comment has been minimized.
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
2 times, most recently
from
October 21, 2024 14:03
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 |
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 23, 2024 01:15
0adeaf3 to
f8cdf3c
Compare
Member
Author
|
Rebased to the latest master since #131871 is merged just now. |
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 23, 2024 01:22
f8cdf3c to
294a808
Compare
SpriteOvO
force-pushed
the
riscv-int-arg-attr
branch
from
October 23, 2024 02:42
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