Skip to content

[FIX,LOWERING] Add attrs from Relay Functions to PrimFuncs in Metaschedule Lowering#13034

Merged
tkonolige merged 1 commit intoapache:mainfrom
tkonolige:fix_labelops_metaschedule
Oct 11, 2022
Merged

[FIX,LOWERING] Add attrs from Relay Functions to PrimFuncs in Metaschedule Lowering#13034
tkonolige merged 1 commit intoapache:mainfrom
tkonolige:fix_labelops_metaschedule

Conversation

@tkonolige
Copy link
Copy Markdown
Contributor

Attrs were not propogated from Relay functions to the corresponding PrimFunc when lowering with MetaSchedule enabled. These attrs are not seperately copied in the MetaSchedule lowering flow.

@junrushao

…edule Lowering

Attrs were not propogated from Relay functions to the corresponding
PrimFunc when lowering with MetaSchedule enabled. These attrs are not
seperately copied in the MetaSchedule lowering flow.
@tkonolige tkonolige force-pushed the fix_labelops_metaschedule branch from 2596b8f to 144af83 Compare October 11, 2022 16:06
Copy link
Copy Markdown
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.

LGTM. Is there any specific attribute in your mind that needs cross-IR propagation? hash?

@tkonolige
Copy link
Copy Markdown
Contributor Author

Hash is the most important one, but layout information is also useful.

@junrushao
Copy link
Copy Markdown
Member

Does Relay do any follow-up layout optimization after being lowered to TIR?

@tkonolige tkonolige merged commit df0ffc3 into apache:main Oct 11, 2022
@tkonolige tkonolige deleted the fix_labelops_metaschedule branch October 11, 2022 19:30
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
…edule Lowering (apache#13034)

Attrs were not propogated from Relay functions to the corresponding
PrimFunc when lowering with MetaSchedule enabled. These attrs are not
seperately copied in the MetaSchedule lowering flow.
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