This is an archive of the discontinued LLVM Phabricator instance.

[flang] Lower PAUSE statement
ClosedPublic

Authored by clementval on Feb 1 2022, 5:52 AM.

Details

Summary

Lower the PAUSE statement to a runtime call.

This patch is part of the upstreaming effort from fir-dev branch.

Co-authored-by: Eric Schweitz <eschweitz@nvidia.com>
Co-authored-by: Jean Perier <jperier@nvidia.com>

Diff Detail

Event Timeline

clementval created this revision.Feb 1 2022, 5:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 1 2022, 5:52 AM
Herald added a subscriber: mehdi_amini. · View Herald Transcript
clementval requested review of this revision.Feb 1 2022, 5:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 1 2022, 5:52 AM

Could you check why the patch application fails?

flang/test/Lower/pause-statement.f90
2

Nit: is canonicalize required?

clementval added inline comments.Feb 1 2022, 6:25 AM
flang/test/Lower/pause-statement.f90
2

Does not make a difference on this particular test.

clementval updated this revision to Diff 404913.Feb 1 2022, 6:39 AM

Update test

Could you check why the patch application fails?

Probably too late to add the dependency. Should be fine after the rebase.

clementval marked an inline comment as done.Feb 1 2022, 6:46 AM
This revision is now accepted and ready to land.Feb 1 2022, 8:16 AM
schweitz accepted this revision.Feb 1 2022, 9:34 AM
This revision was automatically updated to reflect the committed changes.