ganler commented on pull request #10172:
URL: https://github.com/apache/tvm/pull/10172#issuecomment-1031072485


   @lazycal  Using this impl in fixes the bug you mentioned and might be more 
general to overcome other hidden ones if we can assume that `base` and `stride` 
must be of integers.
   
   ```c++
   Ramp::Ramp(PrimExpr base, PrimExpr stride, int lanes, Span span) {
     ICHECK(base.defined());
     ICHECK(stride.defined());
     ICHECK(base.dtype().is_scalar());
     ICHECK(stride.dtype().is_scalar());
     ICHECK_GT(lanes, 1);
     ICHECK(base.dtype().is_int());
     ICHECK(stride.dtype().is_int());
     
     if (base.dtype() != stride.dtype()) {
       size_t bits = std::max(base.dtype().bits(), stride.dtype().bits());
       DataType dtype = base.dtype().with_bits(bits);
       if (base.dtype() != dtype) base = cast(dtype, base);
       if (stride.dtype() != dtype) stride = cast(dtype, stride);
     }
   
     ObjectPtr<RampNode> node = make_object<RampNode>();
     node->dtype = base.dtype().with_lanes(lanes);
     node->base = base;
     node->stride = stride;
     node->lanes = lanes;
     node->span = std::move(span);
     data_ = std::move(node);
   }
   ```


-- 
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]


Reply via email to