[PATCH] D88154: Initial support for vectorization using Libmvec (GLIBC vector math library).
Sanjay Patel via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 12 13:14:25 PDT 2020
spatel added a comment.
In D88154#2325713 <https://reviews.llvm.org/D88154#2325713>, @venkataramanan.kumar.llvm wrote:
> As per review comments from Sanjay, updated the test case to use metadata. Also autogenerated the checks in the test cases using llvm/utils/update_test_checks.py.
Thanks. But wouldn't it be better test coverage to vary the "llvm.loop.vectorize.width". Why is it always "4"?
I don't have any experience with this lib, so no real feedback on the code itself. Hopefully another reviewer can approve if there are no other concerns.
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D88154/new/
https://reviews.llvm.org/D88154
More information about the llvm-commits
mailing list