Skip to content

Conversation

@avquicinc
Copy link
Contributor

@avquicinc avquicinc commented Aug 12, 2022

Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from Reviewers by @ them in the pull request thread.

cc @mehrdadh

@avquicinc avquicinc changed the title Auto-vectorization (fp16) for v68 [HEXAGON] Auto-vectorization (fp16) for v68 Aug 12, 2022
@avquicinc
Copy link
Contributor Author

Hi @kparzysz-quic,
Created this PR to add the change needed for enabling auto-vectorization for v68.

@masahi
Copy link
Member

masahi commented Aug 17, 2022

@tvm-bot rerun

if not llvm_options:
llvm_options = ""
llvm_options += " -force-hvx-float"

Copy link
Contributor

Choose a reason for hiding this comment

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

Does this flag impose any new requirements regarding which LLVM versions TVM needs?

Copy link
Contributor

Choose a reason for hiding this comment

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

Also, are there any potential downsides or other side-effects of using this flag? Asking because with compiler flags, the word "force" sometimes implies that the resulting behavior isn't always a good idea.

Copy link
Contributor

Choose a reason for hiding this comment

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

It will work with any LLVM version that supports v69. It's fine to use it.

@kparzysz-quic
Copy link
Contributor

@tvm-bot rerun

Let me see if the bot is still confused...
@tvm-bot rerun

@masahi
Copy link
Member

masahi commented Aug 18, 2022

@tvm-bot rerun

@kparzysz-quic kparzysz-quic merged commit 88928a4 into apache:main Aug 18, 2022
@avquicinc avquicinc deleted the v68autovec branch September 1, 2022 19:39
xinetzone pushed a commit to daobook/tvm that referenced this pull request Nov 25, 2022
* Auto-vectorization (fp16) for v68

* use tvm.testing.main in fp16 test of tanh_slice op
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.

4 participants