[flang-commits] [PATCH] D88965: [Flang][OpenMP] Rework parser changes for OpenMP atomic construct.

Kiran Chandramohan via Phabricator via flang-commits flang-commits at lists.llvm.org
Thu Oct 8 13:40:29 PDT 2020


kiranchandramohan accepted this revision.
kiranchandramohan added a comment.
This revision is now accepted and ready to land.

> OmpStructureChecker is supposed to work only with parser::OmpClause
> after tablegen changes for OpenMP and OpenACC were introduced.

I think this was not related to tablegen changes. The semantic checks
only worked with OmpClauses.

> Why do we need unparser tests here, as unparsing would be removed at some point
> in time? So should the tests be removed at that time.

Sorry, I was not aware that they are going to be removed. I guess we can keep it till then.
I would prefer keeping the unparsing since it provides a mechanism to 1) test the parser 
and 2) do source to source translation.

BTW, are allowed clauses and all handled automatically or will they require a separate patch?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D88965/new/

https://reviews.llvm.org/D88965



More information about the flang-commits mailing list