Re: [asan] Another ICE fix

2012-12-03 Thread Dodji Seketeli
Jakub Jelinek ja...@redhat.com writes: Fixed thusly, ok for trunk? 2012-11-27 Jakub Jelinek ja...@redhat.com * asan.c (instrument_assignment): Instrument lhs only for gimple_store_p and rhs1 only for gimple_assign_load_p. This is OK, thanks. And sorry for the delay. --

[asan] Another ICE fix

2012-11-27 Thread Jakub Jelinek
Hi! E.g. on c_char_tests.f03 we have a stmt like _2 = VIEW_CONVERT_EXPRcharacter(kind=1)[1:1](121)[1]{lb: 1 sz: 1}; after inlining (wonder why we never fold that to 121), which asan incorrectly considered to be a load and thus attempted to instrument it, by taking address of the rhs. Fixed