Re: Do not optimize some polymorphic calls with -fsanitize=undefined

2016-04-05 Thread Jan Hubicka
Hi, this is patch I commited to address the feedback. Regtested on x86_64-linux Honza * ipa-devirt.c (maybe_record_node): Fix comment; use SANITIZE_UNREACHABLE instead of SANITIZE_UNDEFINED. * g++.dg/ipa/devirt-51.C: Use -fsanitize=unreachable. Index: ipa-devirt.c

Re: Do not optimize some polymorphic calls with -fsanitize=undefined

2016-04-04 Thread Jan Hubicka
> On Mon, Apr 04, 2016 at 05:50:53PM +0200, Jan Hubicka wrote: > > Hi, > > as requested by Jakub, this patch makes devirtualization code to turn off > > transformations based on assumption that cxa_pure_virtual will never be > > called > > by a virtual call when -fsanitize=undefined is used. > >

Re: Do not optimize some polymorphic calls with -fsanitize=undefined

2016-04-04 Thread Jakub Jelinek
On Mon, Apr 04, 2016 at 05:50:53PM +0200, Jan Hubicka wrote: > Hi, > as requested by Jakub, this patch makes devirtualization code to turn off > transformations based on assumption that cxa_pure_virtual will never be called > by a virtual call when -fsanitize=undefined is used. > >

Do not optimize some polymorphic calls with -fsanitize=undefined

2016-04-04 Thread Jan Hubicka
Hi, as requested by Jakub, this patch makes devirtualization code to turn off transformations based on assumption that cxa_pure_virtual will never be called by a virtual call when -fsanitize=undefined is used. Bootstrapped/regtested x86_64-linux, will commit it shortly. PR ipa/66223