This is an archive of the discontinued LLVM Phabricator instance.

[AArch64] Further enable UnrollAndJam
ClosedPublic

Authored by NickGuy on Jun 3 2021, 3:09 AM.

Details

Summary

Due to the dependency on runtime unrolling, UnJ is only to be enabled by default on in-order scheduling models and if a cpu is specified through -mcpu.

Diff Detail

Event Timeline

NickGuy created this revision.Jun 3 2021, 3:09 AM
NickGuy requested review of this revision.Jun 3 2021, 3:09 AM
Matt added a subscriber: Matt.Jun 3 2021, 8:31 AM

UnJ isn't enabled by default upstream yet in the pass manager, but it should be OK to set the option in the target.

llvm/test/CodeGen/AArch64/aarch64-unroll-and-jam.ll
99

You can probably remove this tbaa info, it doesn't look like there are any places it would help.

NickGuy updated this revision to Diff 349579.Jun 3 2021, 9:11 AM
NickGuy marked an inline comment as done.
dmgreen accepted this revision.Jun 4 2021, 1:21 AM

Thanks. LGTM

This revision is now accepted and ready to land.Jun 4 2021, 1:21 AM
This revision was automatically updated to reflect the committed changes.