This is an archive of the discontinued LLVM Phabricator instance.

[OpenMP] Initial parsing and sema for 'parallel masked' construct
ClosedPublic

Authored by mdfazlay on Jun 9 2022, 4:06 PM.

Diff Detail

Event Timeline

mdfazlay created this revision.Jun 9 2022, 4:06 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 9 2022, 4:06 PM
mdfazlay requested review of this revision.Jun 9 2022, 4:06 PM
Herald added a project: Restricted Project. · View Herald Transcript

Erroneus tests, nesting of regions, analysis tests?

martong removed a subscriber: martong.Jun 10 2022, 4:04 AM
mdfazlay updated this revision to Diff 436589.Jun 13 2022, 3:48 PM

Added erroneous tests, nesting of regions test and analysis tests.

Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 3:48 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
ABataev added inline comments.Jun 14 2022, 7:15 AM
clang/include/clang-c/Index.h
2641

Does this change affect anything else in the compiler?

clang/include/clang/AST/StmtOpenMP.h
2315

final

mdfazlay updated this revision to Diff 436994.Jun 14 2022, 6:47 PM
mdfazlay added inline comments.Jun 14 2022, 6:50 PM
clang/include/clang-c/Index.h
2641

We had to add and keep the CXCursor_OMPParallelMaskedDirective with other OpenMP directives. I believe it doesn't affect anything else in the compiler. We have made necessary changes in "clang/bindings/python/clang/cindex.py" to reflect the new value of CXCursor_TranslationUnit.

clang/include/clang/AST/StmtOpenMP.h
2315

Added in the new diff.

This revision is now accepted and ready to land.Jun 15 2022, 5:54 AM
This revision was landed with ongoing or failed builds.Jun 16 2022, 6:16 PM
This revision was automatically updated to reflect the committed changes.