New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[IR] Refactor op yaml info parser #54790
Conversation
… refactor_op_info
… refactor_op_info
你的PR提交成功,感谢你对开源项目的贡献! |
❌ The PR is not created using PR's template. You can refer to this Demo. |
std::vector<std::pair<std::string, std::string>> inplace; | ||
std::vector<std::pair<std::string, std::string>> view; | ||
OpRunTimeInfo(std::string infer_meta_func, | ||
std::vector<std::string> infer_meta_param, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这里的参数建议都设置为const &
bool no_need_buffer = false; | ||
bool is_mutable_attribute = false; | ||
OpInputInfo() {} | ||
OpInputInfo(const OpInputInfo& input_info) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OpInputInfo(const OpInputInfo& input_info)=default;
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
|
||
const OpInfoTuple& op_info_tuple_; | ||
|
||
const std::vector<OpInputInfo>& vec_op_input_info_; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
这个成员是不是只是op_info_tuple_
第0个位置的vector。这里可以不作为类成员,而是以 private inline const & InputInfo() const
形式来内部使用即可?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done
… refactor_op_info
… refactor_op_info
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
PR types
Others
PR changes
Others
Description
新增Op yaml infor的parser,方便在lowering 和构造context的时候使用,减少重复代码
Other
Pcard-67164