Skip to content

Conversation

@ronlieb
Copy link
Collaborator

@ronlieb ronlieb commented Nov 17, 2025

No description provided.

alexey-bataev and others added 8 commits November 16, 2025 12:39
The compiler should not consider split vectorize nodes, when checking
for non-schedulable PHI-based parent nodes. Only pure PHI nodes must be
  considered, they only can be considered as explicit users, split nodes
  are not.

Fixes llvm#168268
ThisPartition is already of type int.

Identified with readability-redundant-casting.
EnableFSDiscriminator is declared in DebugInfoMetadata.h.

Identified with readability-redundant-declaration.
Update VPInstruction constructor to delegate to constructor with more
comprehensive checking and validation.

This required updating some unit tests, to make sure the constructed
VPInstructions are valid.
This patch moves initWithExactBucketCount and ExactBucketCount to
DenseMapBase to share more code.

Since SmallDenseMap::allocateBuckets always returns true,
initWithExactBucketCount is equivalent to:

  void initWithExactBucketCount(unsigned NewNumBuckets) {
    allocateBuckets(NewNumBuckets);
    initEmpty();
  }

for SmallDenseMap.

Note that ExactBucketCount is not used within DenseMapBase yet.

This moves us closer to the storage policy idea outlined in llvm#168255.
@ronlieb ronlieb requested review from a team and dpalermo November 17, 2025 00:01
@z1-cciauto
Copy link
Collaborator

@z1-cciauto z1-cciauto merged commit 967d5ce into amd-staging Nov 17, 2025
6 checks passed
@z1-cciauto z1-cciauto deleted the amd/merge/upstream_merge_20251116173645 branch November 17, 2025 02:38
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.

7 participants