[PATCH] D119121: [test-suite] Add unit tests for vectorizer memory runtime checks.

[Dave Green via Phabricator via llvm-commits]

dmgreen added a comment.

I tried copying this into a godbolt link, and is suggest you may need to `#include <functional>` under some systems. There are also some warnings that might be worth cleaning up: https://godbolt.org/z/sMTjfKbGo



================
Comment at: SingleSource/UnitTests/Vectorizer/runtime-checks.cpp:208
+  {
+    DEFINE_SCALAR_AND_VECTOR_FN2(_Pragma(
+        "clang loop unroll(disable)") for (unsigned i = 0; i < TC;
----------------
Should this one have this loop unroll Pragma here? It's different to the others.


Repository:
  rT test-suite

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D119121/new/

https://reviews.llvm.org/D119121