This is an archive of the discontinued LLVM Phabricator instance.

[flang][hlfir] Separate -emit-fir and -emit-hlfir for bbc
ClosedPublic

Authored by tblah on May 22 2023, 8:00 AM.

Details

Summary

In review for https://reviews.llvm.org/D146278, @vzakhari asked to
separate -emit-fir and -emit-hlfir. This will allow FIR to be easily
outputted after the HLFIR passes have been run.

The new semantics are as follows:

Action-hlfir?Result
=================================================
-emit-hlfirNOutputs HLFIR
-emit-hlfirYOutputs HLFIR
-emit-firNOutputs FIR, using old lowering
-emit-firYOutputs FIR, lowering via HLFIR

This is tested in flang/test/HLFIR/hlfir-flags.f90

Depends on: D151088

Diff Detail

Event Timeline

tblah created this revision.May 22 2023, 8:00 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptMay 22 2023, 8:00 AM
tblah requested review of this revision.May 22 2023, 8:00 AM
vzakhari accepted this revision.May 22 2023, 12:21 PM

Thank you!

This revision is now accepted and ready to land.May 22 2023, 12:21 PM
jeanPerier accepted this revision.May 23 2023, 12:37 AM

Thank you!

tblah updated this revision to Diff 526682.May 30 2023, 9:46 AM

Updating with new HLFIR tests since last revision

tblah updated this revision to Diff 527388.Jun 1 2023, 6:08 AM

Add new upstream HLFIR test

This revision was landed with ongoing or failed builds.Jun 1 2023, 12:40 PM
This revision was automatically updated to reflect the committed changes.