This is an archive of the discontinued LLVM Phabricator instance.

[Flang][openmp][3/5] Make ProcBind clause part of OmpClause
ClosedPublic

Authored by sameeranjoshi on Dec 21 2020, 9:29 AM.

Details

Summary

After discussion in D93482 we found that the some of the clauses were not
following the common OmpClause convention.

The benefits of using OmpClause:

  • Functionalities from structure checker are mostly aligned to work with llvm::omp::Clause.
  • The unparsing as well can take advantage.
  • Homogeneity with OpenACC and rest of the clauses in OpenMP.
  • Could even generate the parser with TableGen, when there is homogeneity.
  • It becomes confusing when to use flangClass and flangClassValue inside TableGen, if incase we generate parser using TableGen we could have only a single let expression.

This patch makes OmpProcBindClause clause part of OmpClause.
The unparse function is dropped as the unparsing is done by WALK_NESTED_ENUM
for OmpProcBindClause.

Diff Detail

Event Timeline

sameeranjoshi created this revision.Dec 21 2020, 9:29 AM
sameeranjoshi requested review of this revision.Dec 21 2020, 9:29 AM
Herald added a project: Restricted Project. · View Herald Transcript
Herald added a subscriber: sstefan1. · View Herald Transcript
This revision is now accepted and ready to land.Dec 21 2020, 9:37 AM