Mercurial > projects > ldc
annotate gen/optimizer.h @ 1499:df11cdec45a2
Another shot at fixing the issues with (constant) struct literals and their addresses. See DMD2682, #218, #324.
The idea is to separate the notion of const from 'this variable can always be
replaced with its initializer' in the frontend. To do that, I introduced
Declaration::isSameAsInitializer, which is overridden in VarDeclaration to
return false for constants that have a struct literal initializer.
So
{{{
const S s = S(5);
void foo() { auto ps = &s; }
// is no longer replaced by
void foo() { auto ps = &(S(5)); }
}}}
To make taking the address of a struct constant with a struct-initializer
outside of function scope possible, I made sure that AddrExp::optimize doesn't
try to run the argument's optimization with WANTinterpret - that'd again
replace the constant with a struct literal temporary.
author | Christian Kamm <kamm incasoftware de> |
---|---|
date | Sun, 14 Jun 2009 19:49:58 +0200 |
parents | d9c5f5a43403 |
children |
rev | line source |
---|---|
1173
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
1 #ifndef LDC_GEN_OPTIMIZER_H |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
2 #define LDC_GEN_OPTIMIZER_H |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
3 |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
4 namespace llvm { class Module; } |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
5 |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
6 bool ldc_optimize_module(llvm::Module* m); |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
7 |
1482
d9c5f5a43403
Run `semantic3` on imported modules, and emit new symbols with
Frits van Bommel <fvbommel wxs.nl>
parents:
1174
diff
changeset
|
8 // Determines whether the inliner will run in the -O<N> list of passes |
1173
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
9 bool doInline(); |
1482
d9c5f5a43403
Run `semantic3` on imported modules, and emit new symbols with
Frits van Bommel <fvbommel wxs.nl>
parents:
1174
diff
changeset
|
10 // Determines whether the inliner will be run at all. |
d9c5f5a43403
Run `semantic3` on imported modules, and emit new symbols with
Frits van Bommel <fvbommel wxs.nl>
parents:
1174
diff
changeset
|
11 bool willInline(); |
1173
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
12 |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
13 int optLevel(); |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
14 |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
15 bool optimize(); |
b0f9652f31de
Forgot to commit this file, sorry
Frits van Bommel <fvbommel wxs.nl>
parents:
diff
changeset
|
16 |
1174
0b26cfb2d445
Add newlines to end of file.
Christian Kamm <kamm incasoftware de>
parents:
1173
diff
changeset
|
17 #endif |
0b26cfb2d445
Add newlines to end of file.
Christian Kamm <kamm incasoftware de>
parents:
1173
diff
changeset
|
18 |