yongwww commented on a change in pull request #4312: URL: https://github.com/apache/incubator-tvm/pull/4312#discussion_r436280419
########## File path: include/tvm/relay/attrs/transform.h ########## @@ -210,14 +210,22 @@ struct SplitAttrs : public tvm::AttrsNode<SplitAttrs> { /*! \brief Attributes for StridedSlice operator */ struct StridedSliceAttrs : public tvm::AttrsNode<StridedSliceAttrs> { - Array<Integer> begin; - Array<Integer> end; - Array<Integer> strides; + Optional<Array<Integer>> begin; + Optional<Array<Integer>> end; + Optional<Array<Integer>> strides; + bool slice_mode; TVM_DECLARE_ATTRS(StridedSliceAttrs, "relay.attrs.StridedSliceAttrs") { TVM_ATTR_FIELD(begin).describe("Indices for begin of slice, begin index is also inclusive"); TVM_ATTR_FIELD(end).describe("Indices for end of slice, end index is exclusive"); - TVM_ATTR_FIELD(strides).set_default(Array<Integer>({})).describe("Stride values of the slice"); + TVM_ATTR_FIELD(strides).describe("Stride values of the slice"); + TVM_ATTR_FIELD(slice_mode) Review comment: > > We can still call it `slice_mode`, but change it from bool to string option selected from ["end", "size"]. How about this? > > sounds good to me Changed. ---------------------------------------------------------------- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org