Skip to content

Conversation

@z1-cciauto
Copy link
Collaborator

No description provided.

mtrofin and others added 8 commits November 5, 2025 16:57
Revealed in PR llvm#166102, which itself doesn't _cause_ the profile being
dropped. Referencing if it makes debugging easier.
…ead exits (llvm#164714)

As noted in the doc comment of `handleDeadExits`​, the dummy switch is just an artifact of the constraints placed by the fact that we operate in a loop pass. Adding weights here is unnecessary, but the complexity is low, and it helps keep things easy for profile propagation verification (in a sense, the overall complexity would be higher if we special-cased this somehow).

Issue llvm#147390
This was very hackily patched in
16ef893 to not use env -u. The internal
shell does not support unset, but does supprt env -u. Disable the test
for now so we can enable the internal shell with a TODO to enable it
after the internal shell landing has stuck.

Reviewers: fmayer, vitalybuka, w2yehia, daltenty, mingmingl-llvm, madanial0

Reviewed By: mingmingl-llvm

Pull Request: llvm#166637
This fixes a regression exposed after
4454152.
This introduces a few small regressions for true16. There are more cases
where the value can propagate through subregister extracts which need
new handling. They're also small enough that perhaps there's a way to
avoid needing to deal with this case in the first place.
Removing skip after confirming builds pass locally. Upgraded workers to
clang 20

Fixes llvm#162516
@z1-cciauto z1-cciauto requested a review from a team November 6, 2025 04:07
@z1-cciauto
Copy link
Collaborator Author

@ranapratap55
Copy link

@ronlieb
Copy link
Collaborator

ronlieb commented Nov 6, 2025

@ronlieb ronlieb merged commit f6a44f4 into amd-staging Nov 6, 2025
12 of 13 checks passed
@ronlieb ronlieb deleted the upstream_merge_202511052306 branch November 6, 2025 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants