Page MenuHomePhabricator

[flang] [openmp] Correct OpenMP Allocate Directive parse tree structure to associate directives with allocate statements correctly.
Needs ReviewPublic

Authored by I_Perry on Jun 16 2021, 8:40 AM.

Details

Summary

This patch implements a parse tree rewrite such that openmp allocate directives which should be associated with fortran allocate statements are done so.

Diff Detail

Unit TestsFailed

TimeTest
140 msx64 debian > Clang.Format::struct-array-initializer.cpp
Script: -- : 'RUN: at line 1'; grep -Ev "// *[A-Z-]+:" /var/lib/buildkite-agent/builds/llvm-project/clang/test/Format/struct-array-initializer.cpp | /var/lib/buildkite-agent/builds/llvm-project/build/bin/clang-format -style="{BasedOnStyle: LLVM, AlignArrayOfStructures: Right}" /var/lib/buildkite-agent/builds/llvm-project/clang/test/Format/struct-array-initializer.cpp | /var/lib/buildkite-agent/builds/llvm-project/build/bin/FileCheck -strict-whitespace -check-prefix=CHECK1 /var/lib/buildkite-agent/builds/llvm-project/clang/test/Format/struct-array-initializer.cpp

Event Timeline

I_Perry created this revision.Jun 16 2021, 8:40 AM
I_Perry requested review of this revision.Jun 16 2021, 8:40 AM
Herald added a project: Restricted Project. · View Herald Transcript
I_Perry updated this revision to Diff 352682.Jun 17 2021, 4:58 AM

Added test to stop unambiguous start of executable allocate from being ignored.

Can this be in the Canonicalize OpenMP file?
flang/lib/Semantics/canonicalize-omp.cpp

I_Perry updated this revision to Diff 352995.Jun 18 2021, 6:37 AM

Addressed comment to move changes to canonicalize-omp.cpp

I_Perry updated this revision to Diff 352996.Jun 18 2021, 6:39 AM

Removed included <iterator> in rewrite-parse-tree.cpp