https://github.com/tblah commented:

The overall approach LGTM so long as @kiranchandramohan is happy.

Have you looked at the new canonical loop operation `omp.new_cli` (I think it 
was merged since you started this patch)? It isn't obvious if we should allow 
tile+unroll (and the OpenMP 6.0 additions) along with simd. I would guess yes 
but I think matching clang is most important.

https://github.com/llvm/llvm-project/pull/150269
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to