This is an archive of the discontinued LLVM Phabricator instance.

[flang][NFC] Remove obsolete IntrinsicCall helper
ClosedPublic

Authored by clementval on Jan 28 2022, 6:04 AM.

Details

Summary

Remove obsolete code that has moved to the
flang/Optimizer/Builder/Intrinsic directory.

genMin is inlined in the code since it's not available
in the builder.

Diff Detail

Event Timeline

clementval created this revision.Jan 28 2022, 6:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2022, 6:04 AM
clementval requested review of this revision.Jan 28 2022, 6:04 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 28 2022, 6:04 AM

LGTM.

Nit: Can you add a comment that the genMin function is being inlined here and this is not a functionality change?

This revision is now accepted and ready to land.Jan 28 2022, 6:29 AM

I mean can you add that info to the commit message (not as a comment)?

clementval edited the summary of this revision. (Show Details)Jan 28 2022, 6:37 AM

I mean can you add that info to the commit message (not as a comment)?

Sure I updated the description. CharacterExpr.cpp will be removed as well anyway.

schweitz accepted this revision.Jan 28 2022, 9:56 AM
clementval edited the summary of this revision. (Show Details)

Rebase

This revision was landed with ongoing or failed builds.Jan 28 2022, 1:38 PM
This revision was automatically updated to reflect the committed changes.
flang/lib/Lower/CharacterExpr.cpp