Skip to content

Update to LLVM 17#114048

Merged
bors merged 8 commits intorust-lang:masterfrom
nikic:llvm-17
Aug 8, 2023
Merged

Update to LLVM 17#114048
bors merged 8 commits intorust-lang:masterfrom
nikic:llvm-17

Conversation

@nikic
Copy link
Contributor

@nikic nikic commented Jul 25, 2023

Expected LLVM 17.0.0 release date: Sep 5th
Rust 1.73 release date: Oct 5th

Compatibility changes in this PR:

  • Drop LLVM_RUSTLLVM check for target-cpu table, which no longer requires a patch with LLVM 17.
  • Update powerpc data layouts, which now include function alignment information. As usual, downgrade for older LLVM versions.
  • Adjust the stack-protector.rs test so that the stack smashing does not get optimized away.
  • Adjust path of crtbegin.c and crtend.c in compiler-rt.
  • Updated dist-riscv64-linux to use binutils 2.36 in order to recognize the zicsr feature, which is no longer part of the base ISA.
  • Fixed symlink for asm include directory on dist-various-2. We should use /usr/include/x86_64-linux-gnu/asm for the host, rather than /usr/include/asm-generic.

Upstream patches:

Perf run: https://perf.rust-lang.org/compare.html?start=f239bb6bea94d16d902c36d72b5cabdddefb3cab&end=8030d71a95a3ea79f5fc95232c32f9b78effb92d&stat=instructions:u

Fixes #109671.

Successful: dist-x86_64-linux, dist-aarch64-linux, dist-s390x-linux, dist-powerpc-linux, armhf-gnu, wasm32

@rustbot rustbot added 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 Jul 25, 2023
@rustbot
Copy link
Collaborator

rustbot commented Jul 25, 2023

These commits modify compiler targets.
(See the Target Tier Policy.)

@nikic
Copy link
Contributor Author

nikic commented Jul 25, 2023

@bors try @rust-timer queue

@rust-timer

This comment has been minimized.

@rustbot rustbot added the S-waiting-on-perf Status: Waiting on a perf run to be completed. label Jul 25, 2023
@bors
Copy link
Collaborator

bors commented Jul 25, 2023

⌛ Trying commit 26d460f1909c8fc5f29b68fc170531cc70668a89 with merge 7771f7d16a0851edef19de472fea2b17e0179579...

@rust-log-analyzer

This comment has been minimized.

@bors
Copy link
Collaborator

bors commented Jul 25, 2023

💔 Test failed - checks-actions

@bors bors 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 Jul 25, 2023
@klensy
Copy link
Contributor

klensy commented Jul 25, 2023

CMake 3.20.0 or higher is required. You are running version 3.17.5

but was bumped in #113714?

Can this interfere with next cmake build in cmake.sh?

RUN mkdir -p /rustroot/bin && ln -s /usr/bin/cmake3 /rustroot/bin/cmake

@Kobzol
Copy link
Member

Kobzol commented Jul 25, 2023

Probably #113931 should be merged first to make sure that all build dependencies are fine with Ubuntu 22.04 (and no manual CMake installation).

@rustbot rustbot added A-testsuite Area: The testsuite used to check the correctness of rustc T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue. labels Jul 25, 2023
@nikic
Copy link
Contributor Author

nikic commented Jul 25, 2023

@bors try

@bors
Copy link
Collaborator

bors commented Jul 25, 2023

⌛ Trying commit 6c043155b18366d88d87d888abb8c9efb562c8b8 with merge bc5850257ce53374866a70b2ba693ff4b64a0a5f...

@rust-log-analyzer

This comment has been minimized.

@rust-log-analyzer

This comment has been minimized.

@nikic
Copy link
Contributor Author

nikic commented Jul 25, 2023

@bors try

@bors
Copy link
Collaborator

bors commented Jul 25, 2023

⌛ Trying commit 6be1ee2771bd2c51d2ccdaf05819a458d1124695 with merge 3c3d525b34cf1491d7ff1e217aa1d892d51f0102...

@rust-log-analyzer

This comment has been minimized.

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. A-testsuite Area: The testsuite used to check the correctness of rustc merged-by-bors This PR was explicitly merged by bors. relnotes Marks issues that should be documented in the release notes of the next release. S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-bootstrap Relevant to the bootstrap subteam: Rust's build system (x.py and src/bootstrap) T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. T-infra Relevant to the infrastructure team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

LLVM 17 regression: tests/ui/abi/stack-protector.rs no longer actually attempts stack-smashing

10 participants