blob: 220bd9e5208cbb77bf6386af8aebab3592e26e33 [file] [log] [blame]
Chris Lattner086c0142006-02-03 06:21:43 +00001Target Independent Opportunities:
2
3===-------------------------------------------------------------------------===
4
5FreeBench/mason contains code like this:
6
7static p_type m0u(p_type p) {
8 int m[]={0, 8, 1, 2, 16, 5, 13, 7, 14, 9, 3, 4, 11, 12, 15, 10, 17, 6};
9 p_type pu;
10 pu.a = m[p.a];
11 pu.b = m[p.b];
12 pu.c = m[p.c];
13 return pu;
14}
15
16We currently compile this into a memcpy from a static array into 'm', then
17a bunch of loads from m. It would be better to avoid the memcpy and just do
18loads from the static array.
19
20===-------------------------------------------------------------------------===
21
22Get the C front-end to expand hypot(x,y) -> llvm.sqrt(x*x+y*y) when errno and
23precision don't matter (ffastmath). Misc/mandel will like this. :)
24
Chris Lattner086c0142006-02-03 06:21:43 +000025//===---------------------------------------------------------------------===//
26
27Solve this DAG isel folding deficiency:
28
29int X, Y;
30
31void fn1(void)
32{
33 X = X | (Y << 3);
34}
35
36compiles to
37
38fn1:
39 movl Y, %eax
40 shll $3, %eax
41 orl X, %eax
42 movl %eax, X
43 ret
44
45The problem is the store's chain operand is not the load X but rather
46a TokenFactor of the load X and load Y, which prevents the folding.
47
48There are two ways to fix this:
49
501. The dag combiner can start using alias analysis to realize that y/x
51 don't alias, making the store to X not dependent on the load from Y.
522. The generated isel could be made smarter in the case it can't
53 disambiguate the pointers.
54
55Number 1 is the preferred solution.
56
57//===---------------------------------------------------------------------===//
58
Chris Lattner5946fef2006-02-15 19:52:06 +000059DAG combine this into mul A, 8:
60
61int %test(int %A) {
62 %B = mul int %A, 8 ;; shift
63 %C = add int %B, 7 ;; dead, no demanded bits.
64 %D = and int %C, -8 ;; dead once add is gone.
65 ret int %D
66}
67
68This sort of thing occurs in the alloca lowering code and other places that
69are generating alignment of an already aligned value.
70
Chris Lattnera1532bc2006-02-21 18:29:44 +000071//===---------------------------------------------------------------------===//
72
73Turn this into a signed shift right in instcombine:
74
75int f(unsigned x) {
76 return x >> 31 ? -1 : 0;
77}
78
79http://gcc.gnu.org/bugzilla/show_bug.cgi?id=25600
80http://gcc.gnu.org/ml/gcc-patches/2006-02/msg01492.html
81
Chris Lattner89188a12006-03-02 22:34:38 +000082//===---------------------------------------------------------------------===//
83
84We should reassociate:
85int f(int a, int b){ return a * a + 2 * a * b + b * b; }
86into:
87int f(int a, int b) { return a * (a + 2 * b) + b * b; }
88to eliminate a multiply.
Chris Lattnerb27b69f2006-03-04 01:19:34 +000089
90//===---------------------------------------------------------------------===//
91
92On targets with expensive 64-bit multiply, we could LSR this:
93
94for (i = ...; ++i) {
95 x = 1ULL << i;
96
97into:
98 long long tmp = 1;
99 for (i = ...; ++i, tmp+=tmp)
100 x = tmp;
101
102This would be a win on ppc32, but not x86 or ppc64.
103
Chris Lattnerad019932006-03-04 08:44:51 +0000104//===---------------------------------------------------------------------===//
Chris Lattner5b0fe7d2006-03-05 20:00:08 +0000105
106Shrink: (setlt (loadi32 P), 0) -> (setlt (loadi8 Phi), 0)
107
108//===---------------------------------------------------------------------===//
Chris Lattner549f27d22006-03-07 02:46:26 +0000109
110Reassociate is missing this:
111
112int test(int X, int Y) {
113 return (X+X+Y+Y); // (X+Y) << 1;
114}
115
116it needs to turn the shifts into multiplies to get it.
117
118//===---------------------------------------------------------------------===//
119
Chris Lattnerc20995e2006-03-11 20:17:08 +0000120Reassociate should turn: X*X*X*X -> t=(X*X) (t*t) to eliminate a multiply.
121
122//===---------------------------------------------------------------------===//
123
Chris Lattner74cfb7d2006-03-11 20:20:40 +0000124Interesting? testcase for add/shift/mul reassoc:
125
126int bar(int x, int y) {
127 return x*x*x+y+x*x*x*x*x*y*y*y*y;
128}
129int foo(int z, int n) {
130 return bar(z, n) + bar(2*z, 2*n);
131}
132
133//===---------------------------------------------------------------------===//
134
Chris Lattner82c78b22006-03-09 20:13:21 +0000135These two functions should generate the same code on big-endian systems:
136
137int g(int *j,int *l) { return memcmp(j,l,4); }
138int h(int *j, int *l) { return *j - *l; }
139
140this could be done in SelectionDAGISel.cpp, along with other special cases,
141for 1,2,4,8 bytes.
142
143//===---------------------------------------------------------------------===//
144