Skip to content

Conversation

@masahi
Copy link
Member

@masahi masahi commented Mar 3, 2022

This not only improves code hygiene, it also makes it easier to navigate to the function definition inside an editor. Please follow this style for future code.

@junrushao1994 @zxybazh @Hzfengsy

@masahi masahi changed the title [MetaSchedule] Expose CreatePrimFuncFromOutputs in a header and call it directly [Refactor] Expose CreatePrimFuncFromOutputs in a header and call it directly Mar 3, 2022
@masahi masahi marked this pull request as ready for review March 3, 2022 20:07
@masahi masahi changed the title [Refactor] Expose CreatePrimFuncFromOutputs in a header and call it directly [Refactor] Expose meta-schedule related packed func in a header and call it directly Mar 3, 2022
@masahi
Copy link
Member Author

masahi commented Mar 4, 2022

@junrushao1994 good to go?

Copy link
Member

@junrushao junrushao left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @masahi! It's definitely better to expose and directly use C++ APIs in C++-to-C++ calling. Just a few nitpicks then we are good to go!

@junrushao junrushao merged commit 86ba517 into apache:main Mar 4, 2022
@junrushao
Copy link
Member

Thanks @masahi! It’s merged!

ziqiangxu8457 pushed a commit to ziqiangxu8457/tvm that referenced this pull request Mar 6, 2022
…all it directly (apache#10470)

* [MetaSchedule] Expose CreatePrimFuncFromOutputs in a header and call
it directly

* add include guard

* exposed ContextQueryInsideWithScope too

* oops

* add tir namespace for clarity

* address comment
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
…all it directly (apache#10470)

* [MetaSchedule] Expose CreatePrimFuncFromOutputs in a header and call
it directly

* add include guard

* exposed ContextQueryInsideWithScope too

* oops

* add tir namespace for clarity

* address comment
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants