This is an archive of the discontinued LLVM Phabricator instance.

[gicombiner] Import tryCombineIndexedLoadStore()
ClosedPublic

Authored by dsanders on Oct 17 2019, 5:28 PM.

Details

Summary

Now that arbitrary data is supported, import tryCombineIndexedLoadStore()

Depends on D69147

Diff Detail

Event Timeline

dsanders created this revision.Oct 17 2019, 5:28 PM
volkan accepted this revision.Nov 5 2019, 8:44 PM

LGTM.

This revision is now accepted and ready to land.Nov 5 2019, 8:44 PM
This revision was automatically updated to reflect the committed changes.

Hi,

I am investigating a recurring regression on arm-linux bots and bisecting the commits on the build [1] that introduced the regression it points to this one. By reverting I can build the stage1 (I am testing now the stage2). I seems to only happen on arm-linux bots, I didn't see it on aarch64-linux ones.

[1] http://lab.llvm.org:8011/builders/clang-cmake-armv7-selfhost/builds/3251