Re: [PATCH] Fix -fsanitize=undefined -flto (PR sanitizer/60535)

2014-03-18 Thread Richard Biener
On Mon, 17 Mar 2014, Jakub Jelinek wrote: Hi! Apparently rest_of_decl_compilation only calls varpool_finalize_decl if not in_lto_p, so this patch calls it explicitly after that call to make sure with -flto we register the newly created vars with varpool as well. Additionally, the patch

Re: [PATCH] Fix -fsanitize=undefined -flto (PR sanitizer/60535)

2014-03-18 Thread Jakub Jelinek
On Tue, Mar 18, 2014 at 10:00:09AM +0100, Richard Biener wrote: Isn't varpool_finalize_decl the canonical interface anyway? Thus, you don't need to call rest_of_decl_compilation AFAIK - the varpool machinery will do that. ... Should be enough to check if int128_integer_type_node is not NULL.

[PATCH] Fix -fsanitize=undefined -flto (PR sanitizer/60535)

2014-03-17 Thread Jakub Jelinek
Hi! Apparently rest_of_decl_compilation only calls varpool_finalize_decl if not in_lto_p, so this patch calls it explicitly after that call to make sure with -flto we register the newly created vars with varpool as well. Additionally, the patch gives name to a few further builtin types, so that