This is an archive of the discontinued LLVM Phabricator instance.

[mlir:LLVM] Rudimentary inlining support for LLVM load store.
ClosedPublic

Authored by definelicht on Jan 6 2023, 1:21 AM.

Details

Summary

Conservatively only allow inlining for loads and stores that don't carry
any attributes that require handling while inlining. This can later be
relaxed when proper handling is introduced.

Diff Detail

Event Timeline

definelicht created this revision.Jan 6 2023, 1:21 AM
Herald added a project: Restricted Project. · View Herald Transcript
definelicht requested review of this revision.Jan 6 2023, 1:21 AM
gysit accepted this revision.Jan 6 2023, 2:36 AM

LGTM!

This revision is now accepted and ready to land.Jan 6 2023, 2:36 AM

Move inlined functions above the function they are being inlined into.

Dinistro accepted this revision.Jan 6 2023, 7:53 AM

LGTM!

This revision was automatically updated to reflect the committed changes.