diff options
| author | Juergen Ributzka <juergen@apple.com> | 2014-08-13 22:08:02 +0000 |
|---|---|---|
| committer | Juergen Ributzka <juergen@apple.com> | 2014-08-13 22:08:02 +0000 |
| commit | 7cee768e55ca9fd67859e0794e3906e0eb5a0623 (patch) | |
| tree | 3074aece39a1ad87df88ccf56741c0495a0116bc /llvm/test/CodeGen/ARM/fast-isel-call.ll | |
| parent | fe2c11ffd6b0ac7686ac99ad5e53e6a8e3dfe151 (diff) | |
| download | bcm5719-llvm-7cee768e55ca9fd67859e0794e3906e0eb5a0623.tar.gz bcm5719-llvm-7cee768e55ca9fd67859e0794e3906e0eb5a0623.zip | |
[FastISel] Let the target decide first if it wants to materialize a constant.
This changes the order in which FastISel tries to materialize a constant.
Originally it would try to use a simple target-independent approach, which
can lead to the generation of inefficient code.
On X86 this would result in the use of movabsq to materialize any 64bit
integer constant - even for simple and small values such as 0 and 1. Also
some very funny floating-point materialization could be observed too.
On AArch64 it would materialize the constant 0 in a register even the
architecture has an actual "zero" register.
On ARM it would generate unnecessary mov instructions or not use mvn.
This change simply changes the order and always asks the target first if it
likes to materialize the constant. This doesn't fix all the issues
mentioned above, but it enables the targets to implement such
optimizations.
Related to <rdar://problem/17420988>.
llvm-svn: 215588
Diffstat (limited to 'llvm/test/CodeGen/ARM/fast-isel-call.ll')
| -rw-r--r-- | llvm/test/CodeGen/ARM/fast-isel-call.ll | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/llvm/test/CodeGen/ARM/fast-isel-call.ll b/llvm/test/CodeGen/ARM/fast-isel-call.ll index 9d9e341647a..74b31bd9e67 100644 --- a/llvm/test/CodeGen/ARM/fast-isel-call.ll +++ b/llvm/test/CodeGen/ARM/fast-isel-call.ll @@ -117,17 +117,11 @@ entry: ; ARM-LONG: blx [[R]] ; THUMB: @t10 ; THUMB: movs [[R0:l?r[0-9]*]], #0 -; THUMB: movt [[R0]], #0 ; THUMB: movs [[R1:l?r[0-9]*]], #248 -; THUMB: movt [[R1]], #0 ; THUMB: movs [[R2:l?r[0-9]*]], #187 -; THUMB: movt [[R2]], #0 ; THUMB: movs [[R3:l?r[0-9]*]], #28 -; THUMB: movt [[R3]], #0 ; THUMB: movw [[R4:l?r[0-9]*]], #40 -; THUMB: movt [[R4]], #0 ; THUMB: movw [[R5:l?r[0-9]*]], #186 -; THUMB: movt [[R5]], #0 ; THUMB: and [[R0]], [[R0]], #255 ; THUMB: and [[R1]], [[R1]], #255 ; THUMB: and [[R2]], [[R2]], #255 |
