Skip to content

Conversation

@z1-cciauto
Copy link
Collaborator

No description provided.

pietroalbini and others added 9 commits October 26, 2025 11:54
Recently switched jobs. In practice this doesn't change much since I'm
still in the security group to represent Rust, but I'm updating the
actual company I work for to keep the list up to date.
…pVF (llvm#156723)

Transform TC and VF to same numerical space when they are different.
A conventional "if" statement is easier to read than the
do-while(false) pattern used here.
The "if" statement being removed in this patch is identical to the
"else" clause.
Note that "override" makes "virtual" redundant.

Identified with modernize-use-override.
Enabling modules makes the CI quite a bit faster with basically no
downsides. The non-modules build is still tested through the
`generic-cxxab` configurations, but most of the other CI runs on
platforms with modules support now use modules.
…tion (llvm#165048)

Typed enums are c23 features and are too new to be used. This PR
restores the types in the `__processor_model` struct back to `unsigned
int`, removes typed enums, and uses the enum in the function as a
variable that's later assigned to a struct in order to prevent errors
fixed initially here: llvm#164713

See llvm#165034 for more background
These are basically synonymous with marking windows as an unsupported
platform at this point and should be removed in favor of such
annotations. Removing the remianing annotations which should further
unblock removing the feature altogether now that everything minus
compiler-rt is using the internal shell by default. These were missed
when making the tests compatible with the internal shell.
@z1-cciauto z1-cciauto requested a review from a team October 26, 2025 19:07
@z1-cciauto
Copy link
Collaborator Author

@ronlieb ronlieb merged commit c1ba43c into amd-staging Oct 26, 2025
13 checks passed
@ronlieb ronlieb deleted the upstream_merge_202510261507 branch October 26, 2025 21:58
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.

9 participants