Skip to content

Commit 47db273

Browse files
committed
Use assert instead of if checking
1 parent 6bffc6b commit 47db273

File tree

1 file changed

+2
-4
lines changed

1 file changed

+2
-4
lines changed

mlir/lib/Conversion/GPUToSPIRV/GPUToSPIRV.cpp

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -435,10 +435,8 @@ LogicalResult GPUShuffleConversion::matchAndRewrite(
435435
return rewriter.notifyMatchFailure(
436436
shuffleOp, "shuffle width and target subgroup size mismatch");
437437

438-
// Ensure the offset is a signless/unsigned integer.
439-
if (adaptor.getOffset().getType().isSignedInteger())
440-
return rewriter.notifyMatchFailure(
441-
shuffleOp, "shuffle offset must be a signless/unsigned integer");
438+
assert(!adaptor.getOffset().getType().isSignedInteger() &&
439+
"shuffle offset must be a signless/unsigned integer");
442440

443441
Location loc = shuffleOp.getLoc();
444442
auto scope = rewriter.getAttr<spirv::ScopeAttr>(spirv::Scope::Subgroup);

0 commit comments

Comments
 (0)