[all-commits] [llvm/llvm-project] 65a1eb: Revert "[flang][fir] Lower `do concurrent` loop ne...

Kareem Ergawy via All-commits all-commits at lists.llvm.org
Tue Apr 15 21:23:27 PDT 2025


  Branch: refs/heads/users/ergawy/revert_132904
  Home:   https://github.com/llvm/llvm-project
  Commit: 65a1ebaae316fea153a5d6e85716cdab33a1be6c
      https://github.com/llvm/llvm-project/commit/65a1ebaae316fea153a5d6e85716cdab33a1be6c
  Author: ergawy <kareem.ergawy at amd.com>
  Date:   2025-04-15 (Tue, 15 Apr 2025)

  Changed paths:
    M flang/lib/Lower/Bridge.cpp
    M flang/lib/Optimizer/Builder/FIRBuilder.cpp
    M flang/test/Lower/do_concurrent.f90
    M flang/test/Lower/do_concurrent_local_default_init.f90
    M flang/test/Lower/loops.f90
    M flang/test/Lower/loops3.f90
    M flang/test/Lower/nsw.f90
    M flang/test/Transforms/DoConcurrent/basic_host.f90
    M flang/test/Transforms/DoConcurrent/locally_destroyed_temp.f90
    M flang/test/Transforms/DoConcurrent/loop_nest_test.f90
    M flang/test/Transforms/DoConcurrent/multiple_iteration_ranges.f90
    M flang/test/Transforms/DoConcurrent/non_const_bounds.f90
    M flang/test/Transforms/DoConcurrent/not_perfectly_nested.f90

  Log Message:
  -----------
  Revert "[flang][fir] Lower `do concurrent` loop nests to `fir.do_concurrent` (#132904)"

This reverts commit 04b87e15e40f8857e29ade8321b8b67691545a50.

The reasons for reverting is that the following:
1. I still need need to upstream some part of the do concurrent to
   OpenMP pass from our downstream implementation and taking this in
   downstream will make things more difficult.
2. I still need to work on a solution for modeling locality specifiers
   on `hlfir.do_concurrent` ops. I would prefer to do that and merge the
   entire stack together instead of having a partial solution.



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list