[AArch64][GlobalISel] Share address mode selection code for memops
authorJessica Paquette <jpaquette@apple.com>
Wed, 9 Sep 2020 16:45:54 +0000 (09:45 -0700)
committerJessica Paquette <jpaquette@apple.com>
Wed, 9 Sep 2020 22:14:46 +0000 (15:14 -0700)
commit480e7f43a22578beaa2edc7a271e77793222a1c3
tree4a034ac6b511a47d99cbf045ece7046c87bcbc92
parent9969c317ff0877ed6155043422c70e1d4c028a35
[AArch64][GlobalISel] Share address mode selection code for memops

We were missing support for the G_ADD_LOW + ADRP folding optimization in the
manual selection code for G_LOAD, G_STORE, and G_ZEXTLOAD.

As a result, we were missing cases like this:

```
@foo = external hidden global i32*
define void @baz(i32* %0) {
store i32* %0, i32** @foo
ret void
}
```

https://godbolt.org/z/16r7ad

This functionality already existed in the addressing mode functions for the
importer. So, this patch makes the manual selection code use
`selectAddrModeIndexed` rather than duplicating work.

This is a 0.2% geomean code size improvement for CTMark at -O3.

There is one code size increase (0.1% on lencod) which is likely because
`selectAddrModeIndexed` doesn't look through constants.

Differential Revision: https://reviews.llvm.org/D87397
llvm/lib/Target/AArch64/GISel/AArch64InstructionSelector.cpp
llvm/test/CodeGen/AArch64/GlobalISel/select-store.mir