Re: [ubsan] Call c_fully_fold on SAVE_EXPRs

2013-07-20 Thread Joseph S. Myers
On Sat, 20 Jul 2013, Marek Polacek wrote: > Ok for ubsan branch? I explicitly have no comments on this branch patch. When there's something for trunk I'll review that patch as a whole. -- Joseph S. Myers jos...@codesourcery.com

[ubsan] Call c_fully_fold on SAVE_EXPRs

2013-07-20 Thread Marek Polacek
With ubsan, we ICEd in gimplify_expr e.g. on the following testcase typedef volatile int VI; int foo (void) { VI vi = 100; return 2 << ++vi; } because c_save_expr created c_maybe_const_expr which then leaked into gimplification, oops. Fixed by calling c_fully_fold on second argument; I'm ca