diff options
| author | Eli Friedman <eli.friedman@gmail.com> | 2008-11-13 02:13:11 +0000 |
|---|---|---|
| committer | Eli Friedman <eli.friedman@gmail.com> | 2008-11-13 02:13:11 +0000 |
| commit | 8553a98b8e53c9c74ca46d57a29c995f736dd46b (patch) | |
| tree | d57ad8a57e1b9503bbf894e581bde9d0b59c8b4c /clang/test/Sema/const-eval.c | |
| parent | df21c6e203cfb9f7a26131ca63cf48ae25037958 (diff) | |
| download | bcm5719-llvm-8553a98b8e53c9c74ca46d57a29c995f736dd46b.tar.gz bcm5719-llvm-8553a98b8e53c9c74ca46d57a29c995f736dd46b.zip | |
Backout of r59196, plus a new ICE test. Sorry if this is a
little rude; I figure it's cleaner to just back this out now so
it doesn't get forgotten or mixed up with other checkins.
The modification to isICE is simply wrong; I've added a test that the
change to isICE breaks.
I'm pretty sure the modification to tryEvaluate is also wrong.
At the very least, there's some serious miscommunication going on here,
as this is going in exactly the opposite direction of r59105. My
understanding is that tryEvaluate is not supposed to care about side
effects. That said, a lot of the clients to tryEvaluate are
expecting it to enforce a no-side-effects policy, so we probably need
another method that provides that guarantee.
llvm-svn: 59212
Diffstat (limited to 'clang/test/Sema/const-eval.c')
| -rw-r--r-- | clang/test/Sema/const-eval.c | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/clang/test/Sema/const-eval.c b/clang/test/Sema/const-eval.c index c808b4818f7..f42ed2504b1 100644 --- a/clang/test/Sema/const-eval.c +++ b/clang/test/Sema/const-eval.c @@ -11,5 +11,3 @@ struct y {int x,y;}; EVAL_EXPR(6, (int)(1+(struct y*)0)) EVAL_EXPR(7, (int)&((struct y*)0)->y) EVAL_EXPR(8, (_Bool)"asdf") -void g0(void); -EVAL_EXPR(9, (g0(), 12)) // expected-error {{fields must have a constant size}} |
