lhutton1 commented on code in PR #16899:
URL: https://github.com/apache/tvm/pull/16899#discussion_r1577524346
##########
src/arith/const_int_bound.cc:
##########
@@ -369,6 +370,8 @@ class ConstIntBoundAnalyzer::Impl
return VisitLeftShift(op);
} else if (op->op.same_as(tir::builtin::bitwise_and())) {
return VisitBitwiseAnd(op);
+ } else if (op->op.same_as(tir::builtin::vscale()) && TargetHasSVE()) {
+ return MakeBound(1, 16);
Review Comment:
nit: we could make the upper bound the length of `kAArch64VScaleValues`
incase more values are added in the future. Happy for this to be added in a
later patch though
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
