Skip to content

Conversation

@masahi
Copy link
Member

@masahi masahi commented Feb 22, 2022

Without this, people might see an error like

free(): invalid pointer
Aborted (core dumped)

This is a simple workaround for PyTorch vs LLVM symbol crash problem, that doesn't involve rebuild.

@junrushao1994 @zxybazh

@junrushao
Copy link
Member

looks like we couldn't import torch directly:

E   ModuleNotFoundError: No module named 'torch'

Copy link
Member

@zxybazh zxybazh left a comment

Choose a reason for hiding this comment

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

LGTM but torch is not a tvm requirement so it might not work in the mainline.

@masahi
Copy link
Member Author

masahi commented Feb 22, 2022

ok added a workaround for ci-cpu (ci-gpu has PyTorch).

This is a workaround on top of another workaround...

Copy link
Member

@zxybazh zxybazh left a comment

Choose a reason for hiding this comment

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

LGTM as long as it could pass CI.

@masahi masahi merged commit 27d9960 into apache:main Feb 22, 2022
pfk-beta pushed a commit to pfk-beta/tvm that referenced this pull request Apr 11, 2022
…est (apache#10342)

* Simple workaround for PyTorch symbol crash problem in meta schedule test

* workaround for CI
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.

3 participants