This is an archive of the discontinued LLVM Phabricator instance.

[OPENMP]Initial support for error directive.
ClosedPublic

Authored by jyu2 on Nov 1 2022, 4:52 PM.

Details

Summary

Initial support for error directive.

Diff Detail

Event Timeline

jyu2 created this revision.Nov 1 2022, 4:52 PM
Herald added a project: Restricted Project. · View Herald Transcript
jyu2 requested review of this revision.Nov 1 2022, 4:52 PM
ABataev added inline comments.Nov 2 2022, 6:37 AM
clang/include/clang/AST/StmtOpenMP.h
6229

final

llvm/include/llvm/Frontend/OpenMP/OMP.td
529

OMP_error

jyu2 updated this revision to Diff 472640.Nov 2 2022, 9:06 AM

Thanks Alexey for the review. This is to address his comments.

clang/include/clang/AST/StmtOpenMP.h
6229

Sorry, forget that. Changed.

llvm/include/llvm/Frontend/OpenMP/OMP.td
529

:-( Thank you so much! Changed

jyu2 updated this revision to Diff 472707.Nov 2 2022, 11:22 AM

Fix clang format problem.

This revision is now accepted and ready to land.Nov 2 2022, 11:51 AM
This revision was automatically updated to reflect the committed changes.