Skip to content

Commit 93c7ba1

Browse files
[TLX][AMD] DotConsumerBackward: comment empty pure-virtual overrides
Note that the empty bodies are intentional for an info-only lattice: unanalyzed cases stay Uninitialized, which findDotConsumer treats the same as Conflict, and the caller falls back to the default encoding. This contrasts with DotRewriteBackward (legality analysis) which has to poison unanalyzed paths. Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent b4e7bf1 commit 93c7ba1

1 file changed

Lines changed: 5 additions & 0 deletions

File tree

third_party/tlx/dialect/lib/Transforms/InsertRequireLayout.cpp

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -523,6 +523,11 @@ class DotConsumerBackward
523523
return success();
524524
}
525525

526+
// Required pure-virtual overrides. This is an info-only lattice (not a
527+
// legality analysis like DotRewriteBackward, which poisons here): leaving
528+
// unanalyzed cases as Uninitialized is safe because findDotConsumer
529+
// returns nullopt for both Uninitialized and Conflict, and the caller
530+
// falls back to buildDefaultTDMDescriptorEncoding.
526531
void visitBranchOperand(OpOperand &) override {}
527532
void visitCallOperand(OpOperand &) override {}
528533
void setToExitState(DotConsumerLattice *) override {}

0 commit comments

Comments
 (0)