[flang-commits] [flang] [llvm] [mlir] [MLIR][Flang][OpenMP] Make omp.simdloop into a loop wrapper (PR #87365)

Sergio Afonso via flang-commits flang-commits at lists.llvm.org
Tue Apr 16 07:44:41 PDT 2024


skatrak wrote:

Thank you @Meinersbur for the comments, they should be addressed now.

I checked the tests and the only ones that had whitespace removal were the two "if-clause.f90" tests. I only saw changes in indentation in the places where now code goes nested one level deeper due to replacing `omp.simdloop` with `omp.simd + omp.loop_nest`, so I think it wouldn't make sense to move these changes to another PR.

If it's ok, I'd prefer to just keep these changes and avoid splitting this patch, since there's a big dependent patch I'd have to update as well. Let me know if I should still split the whitespace removal into its own PR.

https://github.com/llvm/llvm-project/pull/87365


More information about the flang-commits mailing list