[all-commits] [llvm/llvm-project] a6e616: Revert "[flang] OpenMP allocate directive parse tr...

Ethan Luis McDonough via All-commits all-commits at lists.llvm.org
Fri May 5 13:52:36 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a6e616cdb1bb89f19a9df5e72b1e4256fed19968
      https://github.com/llvm/llvm-project/commit/a6e616cdb1bb89f19a9df5e72b1e4256fed19968
  Author: Ethan Luis McDonough <ethanluismcdonough at gmail.com>
  Date:   2023-05-05 (Fri, 05 May 2023)

  Changed paths:
    M flang/lib/Parser/unparse.cpp
    M flang/lib/Semantics/canonicalize-omp.cpp
    M flang/lib/Semantics/resolve-directives.cpp
    R flang/test/Parser/OpenMP/allocate-tree-spec-part.f90
    R flang/test/Parser/OpenMP/allocate-tree.f90

  Log Message:
  -----------
  Revert "[flang] OpenMP allocate directive parse tree fix"

This reverts commit 597d8563cd66f23d857196bf135a0c513115ece2.


  Commit: 42df495114dc75fac4f75babe8f1ed43b15515fc
      https://github.com/llvm/llvm-project/commit/42df495114dc75fac4f75babe8f1ed43b15515fc
  Author: Ethan Luis McDonough <ethanluismcdonough at gmail.com>
  Date:   2023-05-05 (Fri, 05 May 2023)

  Changed paths:
    M flang/lib/Parser/unparse.cpp
    M flang/lib/Semantics/canonicalize-omp.cpp
    M flang/lib/Semantics/resolve-directives.cpp
    A flang/test/Parser/OpenMP/allocate-tree-spec-part.f90
    A flang/test/Parser/OpenMP/allocate-tree.f90

  Log Message:
  -----------
  [flang] OpenMP allocate directive parse tree fix

Addresses the same issue as the following abandoned revision: D104391.

Rewrite leading declarative allocations so they are nested within their respective executable allocate directive

Original:
ExecutionPartConstruct -> OpenMPDeclarativeAllocate
ExecutionPartConstruct -> OpenMPDeclarativeAllocate
ExecutionPartConstruct -> OpenMPExecutableAllocate

After rewriting:
ExecutionPartConstruct -> OpenMPExecutableAllocate
| ExecutionPartConstruct -> OpenMPDeclarativeAllocate
| ExecutionPartConstruct -> OpenMPDeclarativeAllocate

Reviewed By: kiranchandramohan

Differential Revision: https://reviews.llvm.org/D148409

Co-authored-by: Isaac Perry <isaac.perry at arm.com>


Compare: https://github.com/llvm/llvm-project/compare/04f3c5a71e8d...42df495114dc


More information about the All-commits mailing list