This is an archive of the discontinued LLVM Phabricator instance.

[flang] Added basic connect to lower OpenMP constructs
ClosedPublic

Authored by shraiysh on Mar 10 2022, 8:23 AM.

Diff Detail

Event Timeline

shraiysh created this revision.Mar 10 2022, 8:23 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald Transcript
shraiysh requested review of this revision.Mar 10 2022, 8:23 AM

Can you lower a small construct with that so we can have a test?

shraiysh updated this revision to Diff 414392.Mar 10 2022, 8:45 AM

Added testcase.

shraiysh updated this revision to Diff 414393.Mar 10 2022, 8:46 AM

Added missing newline at the end of file. Ready for review now.

Maybe I'm missing something but where is the create<mlir::omp::BarrierOp>?

Maybe I'm missing something but where is the create<mlir::omp::BarrierOp>?

Here it is. It was upstreamed over an year ago :) - https://github.com/llvm/llvm-project/blob/58966dd42bc4af67ff6b580a8cc3d36abba98add/flang/lib/Lower/OpenMP.cpp#L77

Maybe I'm missing something but where is the create<mlir::omp::BarrierOp>?

Here it is. It was upstreamed over an year ago :) - https://github.com/llvm/llvm-project/blob/58966dd42bc4af67ff6b580a8cc3d36abba98add/flang/lib/Lower/OpenMP.cpp#L77

My bad :-)

This revision is now accepted and ready to land.Mar 10 2022, 9:04 AM
This revision was automatically updated to reflect the committed changes.