elvin-n commented on a change in pull request #8897:
URL: https://github.com/apache/tvm/pull/8897#discussion_r701614252
##########
File path: python/tvm/topi/x86/utils.py
##########
@@ -18,9 +18,95 @@
import tvm
-def get_fp32_len():
+def target_has_sse42(target):
Review comment:
sse4.2 is a latest standard of SSE type of instructions and supported in
more processors than sse4/sse3 or sse2. It continues to be used in the latest
Intel edge devices for IoT or low end segments (Atom based). If we come to the
requirement distinguish more precisely, we probably will have to redesign this
part.
as for suggestion to introduce the only function instead several ones - I
consider this as valuable comment and in the the future if we want to check
more features this should done. Not sure that it make sense to do in this PR
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
