This is an archive of the discontinued LLVM Phabricator instance.

[fir] Add fir transformational intrinsic builder
ClosedPublic

Authored by clementval on Nov 23 2021, 12:57 PM.

Details

Summary

This patch adds the builder to generate transformational
intrinsic runtime API calls.

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

Co-authored-by: Jean Perier <jperier@nvidia.com>
Co-authored-by: mleair <leairmark@gmail.com>
Co-authored-by: Kiran Chandramohan <kiran.chandramohan@arm.com>
Co-authored-by: Peter Steinfeld <psteinfeld@nvidia.com>

Diff Detail

Event Timeline

clementval created this revision.Nov 23 2021, 12:57 PM
clementval requested review of this revision.Nov 23 2021, 12:57 PM
Herald added a project: Restricted Project. · View Herald TranscriptNov 23 2021, 12:57 PM

Rebase on top of D114557

rovka accepted this revision.Nov 26 2021, 5:22 AM

LGTM with nit.

flang/lib/Optimizer/Builder/Runtime/Transformational.cpp
90

Should we have one for MatmultDirect as well?

This revision is now accepted and ready to land.Nov 26 2021, 5:22 AM
This revision was automatically updated to reflect the committed changes.
clementval marked an inline comment as done.Nov 29 2021, 1:10 AM
clementval added inline comments.
flang/lib/Optimizer/Builder/Runtime/Transformational.cpp
90

It is not needed by lowering in fir-dev so far so I would prefer to not add it until it is needed.