malharJ added inline comments.

================
Comment at: clang/test/CodeGenCXX/pragma-loop-predicate.cpp:88
+
+#pragma clang loop vectorize_predicate(enable) vectorize_width(1)
+  for (int i = 0; i < Length; i++)
----------------
dmgreen wrote:
> Can you add another test for `#pragma clang loop vectorize(disable) 
> vectorize_predicate(enable)`. My understanding is that it will produce the 
> same as `vectorize_predicate(enable) vectorize_width(1)`
I believe test5( ) above takes care of this ? 

comparing the output of the two (test8 and test5), test8 produces an extra 
metadata : 

```
{!"llvm.loop.vectorize.scalable.enable", i1 false}
```



Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D94779

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to