This is an archive of the discontinued LLVM Phabricator instance.

[flang] add hlfir.no_reassoc definition
ClosedPublic

Authored by jeanPerier on Dec 7 2022, 2:56 AM.

Details

Summary

Same as fir.no_reassoc but accepts hlfir.expr type. It is needed because
FIR operation can only produce FIR types, and I do not want to change
that for now.

Depends on D139519

Diff Detail

Event Timeline

jeanPerier created this revision.Dec 7 2022, 2:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 7 2022, 2:56 AM
Herald added a subscriber: jdoerfert. · View Herald Transcript
jeanPerier requested review of this revision.Dec 7 2022, 2:56 AM
This revision is now accepted and ready to land.Dec 7 2022, 2:58 AM
This revision was landed with ongoing or failed builds.Dec 7 2022, 5:45 AM
This revision was automatically updated to reflect the committed changes.

I wonder if we want to keep https://github.com/llvm/llvm-project/blob/main/flang/docs/HighLevelFIR.md in sync with HLFIROps.td. It may be okay to skip it, if it is a temporary solution.