-
Notifications
You must be signed in to change notification settings - Fork 14k
Set signext or zeroext for integer arguments on RISC-V and LoongArch64
#131955
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
Conversation
0ebbea6 to
3f98b30
Compare
|
Oh thank you for 9ba8053 |
This comment has been minimized.
This comment has been minimized.
3f98b30 to
4bf70dd
Compare
|
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
4bf70dd to
dfec22d
Compare
|
Some changes occurred in tests/codegen/sanitizer cc @rust-lang/project-exploit-mitigations, @rcvalle |
dfec22d to
4d7528f
Compare
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
|
@bors try |
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
|
☀️ Try build successful - checks-actions |
0adeaf3 to
f8cdf3c
Compare
|
Rebased to the latest master since #131871 is merged just now. |
f8cdf3c to
294a808
Compare
294a808 to
57bffe1
Compare
So, looks like we're good. @bors r+ |
…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.
|
The job Click to see the possible cause of the failure (guessed by this bot) |
|
💔 Test failed - checks-actions |
|
@bors retry |
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
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 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