Skip to content

Conversation

heiher
Copy link
Contributor

@heiher heiher commented Sep 3, 2025

Fixes #146143

@rustbot
Copy link
Collaborator

rustbot commented Sep 3, 2025

r? @petrochenkov

rustbot has assigned @petrochenkov.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

@rustbot rustbot added A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Sep 3, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 3, 2025

Some changes occurred in compiler/rustc_codegen_gcc

cc @antoyo, @GuillaumeGomez

Some changes occurred in compiler/rustc_codegen_ssa

cc @WaffleLapkin

@rustbot

This comment has been minimized.

@heiher heiher force-pushed the entry-func-features branch from 04f812b to f964331 Compare September 3, 2025 04:06
@petrochenkov petrochenkov added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 3, 2025
@heiher heiher force-pushed the entry-func-features branch from f964331 to beb0304 Compare September 3, 2025 11:44
AirFortressIlikara added a commit to AOSC-Dev/aosc-os-abbs that referenced this pull request Sep 3, 2025
Link: rust-lang/rust#146144

Signed-off-by: Ilikara <3435193369@qq.com>
@heiher heiher force-pushed the entry-func-features branch from beb0304 to 7f26036 Compare September 3, 2025 14:12
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 3, 2025
@bors

This comment was marked as resolved.

@heiher heiher force-pushed the entry-func-features branch from 7f26036 to f028b9b Compare September 4, 2025 07:06
@rustbot
Copy link
Collaborator

rustbot commented Sep 4, 2025

This PR was rebased onto a different master commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

@petrochenkov
Copy link
Contributor

r=me with the nit #146144 (comment) addressed.
@rustbot author

@rustbot rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Sep 4, 2025
@rustbot
Copy link
Collaborator

rustbot commented Sep 4, 2025

Reminder, once the PR becomes ready for a review, use @rustbot ready.

@heiher heiher force-pushed the entry-func-features branch from f028b9b to 923b892 Compare September 4, 2025 13:57
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Sep 4, 2025
@AirFortressIlikara
Copy link

We compile oma with flags below to avoid lsx:

[DEBUG]: LDFLAGS=-Wl,-O1,--sort-common,--as-needed -Wl,-build-id=sha1 -Wl,-z,relro -Wl,-z,now -fPIC -fPIE -flto -fuse-linker-plugin
[DEBUG]: CFLAGS=-pipe -Wno-error -fstack-protector-strong --param=ssp-buffer-size=4 -fexceptions -fPIC -fPIC -ggdb -O2 -fno-omit-frame-pointer -flto -mabi=lp64d -mstrict-align -march=loongarch64 -mtune=loongarch64 -msimd=none
[DEBUG]: CPPFLAGS=-D_GLIBCXX_ASSERTIONS -U_FORTIFY_SOURCE -D_FORTIFY_SOURCE=3
[DEBUG]: CXXFLAGS=-ggdb -fpermissive -pipe -Wno-error -fstack-protector-strong --param=ssp-buffer-size=4 -fexceptions -fPIC -fPIC -ggdb -O2 -fno-omit-frame-pointer -flto -mabi=lp64d -mstrict-align -march=loongarch64 -mtune=loongarch64 -msimd=none
[DEBUG]: OBJCFLAGS=-pipe -Wno-error -fstack-protector-strong --param=ssp-buffer-size=4 -fexceptions -fPIC -fPIC -ggdb -O2 -fno-omit-frame-pointer -flto -mabi=lp64d -mstrict-align -march=loongarch64 -mtune=loongarch64 -msimd=none
[DEBUG]: OBJCXXFLAGS=-fpermissive -ggdb -fpermissive -pipe -Wno-error -fstack-protector-strong --param=ssp-buffer-size=4 -fexceptions -fPIC -fPIC -ggdb -O2 -fno-omit-frame-pointer -flto -mabi=lp64d -mstrict-align -march=loongarch64 -mtune=loongarch64 -msimd=none
[DEBUG]: RUSTFLAGS=-Ccodegen-units=1 -Copt-level=3 -Cdebuginfo=line-tables-only -Cforce-frame-pointers=yes -Cembed-bitcode=yes -Clinker-plugin-lto -Clinker=clang -Clink-arg=-flto -Clink-arg=-fuse-ld=lld -Clink-arg=-Wl,-build-id=sha1 -Clink-arg=-Wl,--lto-O3 -Ctarget-cpu=generic-la64 -Ctarget-feature=-lsx,+d -Clink-arg=-mabi=lp64

When using rustc 1.89.0, lsx exists
图片
Recompile rustc 1.89.0 with this patch (f028b9b) , lsx no longer exists
图片

This patch works as we expected

AirFortressIlikara added a commit to AOSC-Dev/aosc-os-abbs that referenced this pull request Sep 5, 2025
Link: rust-lang/rust#146144

Signed-off-by: Ilikara <3435193369@qq.com>
@petrochenkov
Copy link
Contributor

@bors r+

@bors
Copy link
Collaborator

bors commented Sep 5, 2025

📌 Commit 923b892 has been approved by petrochenkov

It is now in the queue for this repository.

@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 Sep 5, 2025
bors added a commit that referenced this pull request Sep 5, 2025
Rollup of 11 pull requests

Successful merges:

 - #138944 (Add `__isPlatformVersionAtLeast` and `__isOSVersionAtLeast` symbols)
 - #139113 (unstable book: in a sanitizer example, check the code)
 - #145735 (style-guide: Document absence of trailing whitespace)
 - #146041 (tidy: --bless now makes escheck run with --fix)
 - #146144 (compiler: Apply target features to the entry function)
 - #146225 (Simplify `{f16, f32, f64, f128}::midpoint()`)
 - #146234 (change file-is-generated doc comment to inner)
 - #146241 (rustc_infer: change top-level doc comment to inner)
 - #146242 (Ensure that `--html-after-content` option is used to check `scrape_examples_ice` rustdoc GUI test)
 - #146243 (remove couple of redundant clones)
 - #146250 (Bump stage0 rustfmt)

Failed merges:

 - #146200 (Simplify rustdoc-gui tester by calling directly browser-ui-test)

r? `@ghost`
`@rustbot` modify labels: rollup
@bors bors merged commit 1c2d264 into rust-lang:master Sep 5, 2025
10 checks passed
@rustbot rustbot added this to the 1.91.0 milestone Sep 5, 2025
@bors
Copy link
Collaborator

bors commented Sep 5, 2025

⌛ Testing commit 923b892 with merge 6c699a3...

rust-timer added a commit that referenced this pull request Sep 6, 2025
Rollup merge of #146144 - heiher:entry-func-features, r=petrochenkov

compiler: Apply target features to the entry function

Fixes #146143
@heiher heiher deleted the entry-func-features branch September 6, 2025 01:25
stydxm pushed a commit to AOSC-Dev/aosc-os-abbs that referenced this pull request Sep 7, 2025
Link: rust-lang/rust#146144

Signed-off-by: Ilikara <3435193369@qq.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-LLVM Area: Code generation parts specific to LLVM. Both correctness bugs and optimization-related issues. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Global -Ctarget-feature flags not applied to entry function
7 participants