This is an archive of the discontinued LLVM Phabricator instance.

[flang][NFC] remove spurious dependency from IntrinsicCall 2/6
ClosedPublic

Authored by tblah on Feb 1 2023, 8:16 AM.

Diff Detail

Event Timeline

tblah created this revision.Feb 1 2023, 8:16 AM
Herald added projects: Restricted Project, Restricted Project. · View Herald TranscriptFeb 1 2023, 8:16 AM
tblah requested review of this revision.Feb 1 2023, 8:16 AM

What is the ultimate goal of this? This code seems more related to lowering than optimization. Lowering will have a dependency in the other sense right?

tblah added a comment.Feb 1 2023, 8:58 AM

What is the ultimate goal of this? This code seems more related to lowering than optimization. Lowering will have a dependency in the other sense right?

This is to allow implementation of HLFIR transformational intrinsic operations such as hlfir.sum. There will be a pass which rewrites hlfir.sum into the fir.call (and argument handling). lib/Lower already depends upon lib/Optimizer, so it would be best not to add a dependency from lib/Optimizer to lib/Lower (by leaving IntrinsicCall in lib/Lower and calling these functions from lib/Optimizer/HLFIR/Transforms/).

This is to allow implementation of HLFIR transformational intrinsic operations such as hlfir.sum. There will be a pass which rewrites hlfir.sum into the fir.call (and argument handling). lib/Lower already depends upon lib/Optimizer, so it would be best not to add a dependency from lib/Optimizer to lib/Lower (by leaving IntrinsicCall in lib/Lower and calling these functions from lib/Optimizer/HLFIR/Transforms/).

Ok thanks for the explanation

This revision is now accepted and ready to land.Feb 1 2023, 10:50 AM
This revision was landed with ongoing or failed builds.Feb 6 2023, 2:35 AM
This revision was automatically updated to reflect the committed changes.