Mercurial > projects > ldc
annotate tests/mini/norun_debug3.d @ 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 | 925779c19395 |
children |
rev | line source |
---|---|
416
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
1 module mini.norun_debug3; |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
2 |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
3 void main() |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
4 { |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
5 int i = 42; |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
6 int* ip = &i; |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
7 |
1425
925779c19395
I ran into another issue like the one fixed by r1318, so update all of
Frits van Bommel <fvbommel wxs.nl>
parents:
416
diff
changeset
|
8 int* fail = cast(int*) 1; |
416
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
9 *fail = 0; |
24ea9bf5f8b7
testing the new mini test runner features.
Tomas Lindquist Olsen <tomas.l.olsen@gmail.com>
parents:
diff
changeset
|
10 } |