Re: [PATCH, wide-int] change fixed_wide_int_storage from class to struct

On May 15, 2016 9:03:22 PM GMT+02:00, Gerald Pfeifer wrote: >While not a bug according to the language of the C++ standard, this >causes dozens of warnings when building GCC with clang, and there is >not benefit of mixing struct and class like this. > >Mike, when I had a

Re: [PATCH][RFC] Introduce BIT_FIELD_INSERT

I did run into some ICEs with bootstrap/regtest, though: 26c26 < /home/wschmidt/gcc/build/gcc-mainline-base/gcc/testsuite/g++/../../xg++ version 7.0.0 20160515 (experimental) [trunk revision 236259] (GCC) --- > /home/wschmidt/gcc/build/gcc-mainline-test/gcc/testsuite/g++/../../xg++ >

[RFC] Type promotion pass and elimination of zext/sext

Hi Richard, Now that stage1 is open, I would like to get the type promotion passes reviewed again. I have tested the patches on aarch64, x86-64, and ppc64le without any new execution failures. There some test-cases that fails for patterns. I will address them after getting feedback on the basic

Enable inlining into thunks

Hi, this patch teach inliner to inline into thunks. This is easy to do - all we need is to produce a gimple body when we decide to do so. This fixes some ages old xfails and enables some 40k inlines in Firefox. Not all those inlines are win, because the codst model of thunks is wrong. We need to

Fix ICE while inlining thunks

Hi, when inlining thunks we incorrectly update edge caches of callees. This is because bogus function_symbol call. edge->callee is the inline function body and that is never alias. Bootstrapped/regtste x86_64-linux, will commit it tomorrow. * ipa-inline.c (report_inline_failed_reason):

Fix partitioning of inline thunks

Hi, this patch fixes the partitioner WRT inlined thunks. When thunk is offline, we must place it into the same unit as its target (because ASM thunk output machinery is not designed for non-local calls). However when thunk is inlined it behave like other functions and in particular we do not want

Properly set target/optimization options for thunks in free_lang_data

Hi, this patch updates free lang data to also set target/optimization flags for thunks. This is because we eventually may produce a body for the thunk and it needs to be compiled with correct settings. Bootstrapped/regtested x86_64-linux, will commit it tomorrow. Honza * tree.c

Re: C, C++: New warning for memset without multiply by elt size

For the record, this new warning/code found at least three actual bugs in Wine (two of which were in the testsuite, but still). Definitely a useful addition. Gerald

Re: [PATCH, wide-int] change fixed_wide_int_storage from class to struct

On Sun, May 15, 2016 at 12:03 PM, Gerald Pfeifer wrote: > While not a bug according to the language of the C++ standard, this > causes dozens of warnings when building GCC with clang, and there is > not benefit of mixing struct and class like this. > > Mike, when I had a

[PATCH, wide-int] change fixed_wide_int_storage from class to struct

While not a bug according to the language of the C++ standard, this causes dozens of warnings when building GCC with clang, and there is not benefit of mixing struct and class like this. Mike, when I had a similar case a while ago you sounded pretty supportive of this kind of change. Would you