Hi,
while compiling firefox I noticed that -fprofile-generage -flto goes to 8GB.
It turns out that this is caused by ipa_reference no longer being disabled
becaus in_lto_p became flag that is set later (it is not clear to me why it
needs to be this way).
I however do not see reason why not
On Thu, 17 Apr 2014, Jan Hubicka wrote:
Hi,
while compiling firefox I noticed that -fprofile-generage -flto goes to 8GB.
It turns out that this is caused by ipa_reference no longer being disabled
becaus in_lto_p became flag that is set later (it is not clear to me why it
needs to be this
On Thu, Apr 17, 2014 at 4:30 AM, Jan Hubicka hubi...@ucw.cz wrote:
Hi,
the testcase shows problem where cpp implicit alias is always inline and
symtab_remove_unreachable_nodes removes the body of aliased function before
inlininghappens. The real problem is that cgraph_state is set too early
On Thu, Apr 17, 2014 at 7:30 AM, Jeff Law l...@redhat.com wrote:
On 03/18/14 04:13, bin.cheng wrote:
Hi,
After control flow graph change made by
http://gcc.gnu.org/ml/gcc-patches/2014-02/msg01492.html, case
gcc.dg/tree-ssa/ssa-dom-thread-4.c is broken on logical_op_short_circuit
targets
This fixes running into the exponential value-graph - SLP tree
expansion by artificially limiting the overall SLP tree size.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2014-04-16 Richard Biener rguent...@suse.de
PR tree-optimization/60841
This fixes PR60849 by properly rejecting non-boolean typed
comparisons from valid_gimple_rhs_p so they go through the
gimplification paths.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2014-04-16 Richard Biener rguent...@suse.de
PR middle-end/60849
This fixes PR60836 by emitting a non-proper PHI argument to the
incoming edge.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied to trunk.
Richard.
2014-04-16 Richard Biener rguent...@suse.de
PR tree-optimization/60836
* tree-vect-loop.c
From: Trevor Saunders tsaund...@mozilla.com
Hi,
simplification allowed by previous patch.
bootstrap + regtest passed on x86_64-unknown-linux-gnu, ok?
Trev
2014-03-19 Trevor Saunders tsaund...@mozilla.com
* pass_manager.h (pass_manager::register_dump_files_1): Remove
declaration.
From: Trevor Saunders tsaund...@mozilla.com
Hi,
just removing some dead code.
bootstrapped + regtested against r209414 on x86_64-unknown-linux-gnu, ok?
Trev
2014-03-19 Trevor Saunders tsaund...@mozilla.com
* pass_manager.h (pass_manager::register_dump_files_1): Adjust.
*
From: Trevor Saunders tbsau...@mozilla.com
hi,
its a useful warning, and helps catch bugs in the next two patches.
bootstrap + regtest passed on x86_64-unknown-linux-gnu, ok?
Trev
2014-03-19 Trevor Saunders tsaund...@mozilla.com
* configure.ac: Check for -Woverloaded-virtual and
On Thu, Apr 17, 2014 at 10:37 AM, tsaund...@mozilla.com wrote:
From: Trevor Saunders tsaund...@mozilla.com
Hi,
simplification allowed by previous patch.
bootstrap + regtest passed on x86_64-unknown-linux-gnu, ok?
Ok.
Thanks,
Richard.
Trev
2014-03-19 Trevor Saunders
On Thu, Apr 17, 2014 at 10:37 AM, tsaund...@mozilla.com wrote:
From: Trevor Saunders tsaund...@mozilla.com
Hi,
just removing some dead code.
bootstrapped + regtested against r209414 on x86_64-unknown-linux-gnu, ok?
Ok.
Thanks,
Richard.
Trev
2014-03-19 Trevor Saunders
On Thu, Apr 17, 2014 at 10:37 AM, tsaund...@mozilla.com wrote:
From: Trevor Saunders tbsau...@mozilla.com
hi,
its a useful warning, and helps catch bugs in the next two patches.
bootstrap + regtest passed on x86_64-unknown-linux-gnu, ok?
Ok.
Thanks,
Richard.
Trev
2014-03-19 Trevor
On 16 Apr 2014, at 17:36, Richard Henderson r...@redhat.com wrote:
On 04/16/2014 12:39 AM, Eric Botcazou wrote:
The primary bit of rfc here is the hunk that applies to ada/types.h
with respect to Fat_Pointer. Given that the Ada type, as defined in
s-stratt.ads, does not include alignment, I
On 17 April 2014 01:56, Luke Allardyce wrote:
Thanks, I was wrong about that.
Then I think we should just bite the bullet and provide the new
behaviour. If we do have an abi_tag on those types in the next release
then we can preserve the old behaviour in the old ABI and use the
C++11
On Sat, Dec 7, 2013 at 12:45 AM, Cong Hou co...@google.com wrote:
After further reviewing this patch, I found I don't have to change the
code in tree-vect-stmts.c to allow further type conversion after
widen-mult operation. Instead, I detect the following pattern in
Uros Bizjak ubiz...@gmail.com writes:
On Wed, Apr 16, 2014 at 1:16 PM, Rainer Orth
r...@cebitec.uni-bielefeld.de wrote:
Now that 4.9 has branched, it's time to actually remove the obsolete
Solaris 9 configuration. Most of this is just legwork and falls under
my Solaris maintainership.
A
On Thu, Apr 17, 2014 at 1:30 PM, Jeff Law l...@redhat.com wrote:
On 03/18/14 04:13, bin.cheng wrote:
Hi,
After control flow graph change made by
http://gcc.gnu.org/ml/gcc-patches/2014-02/msg01492.html, case
gcc.dg/tree-ssa/ssa-dom-thread-4.c is broken on logical_op_short_circuit
targets
On Wed, Jul 03, 2013 at 08:14:25AM +0200, Uros Bizjak wrote:
On Tue, Jul 2, 2013 at 10:32 AM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
Bootstrap passing. Updated tests passing on BMI-featured HW.
ChangeLog:
2013-07-02 Kirill Yukhin kirill.yuk...@intel.com
*
On Thu, Apr 17, 2014 at 7:19 AM, Thomas Preud'homme
thomas.preudho...@arm.com wrote:
From: Richard Biener [mailto:richard.guent...@gmail.com]
With handling only the outermost handled-component and then only a
selected subset you'll catch many but not all cases. Why not simply
use
On Tue, 2014-04-15 at 14:24 -0700, Cary Coutant wrote:
+ /* If HOST_WIDE_INT is big enough then represent the bound as
+ a constant value. Note that we need to make sure the type
+ is signed or unsigned. We cannot just add an unsigned
+ constant if the
On Thu, 17 Apr 2014, Richard Biener wrote:
This fixes PR60849 by properly rejecting non-boolean typed
comparisons from valid_gimple_rhs_p so they go through the
gimplification paths.
Could you also accept vector comparisons please?
--
Marc Glisse
Thanks! Sorry, missed that!
K
On Thu, Apr 17, 2014 at 2:13 PM, Jakub Jelinek ja...@redhat.com wrote:
On Wed, Jul 03, 2013 at 08:14:25AM +0200, Uros Bizjak wrote:
On Tue, Jul 2, 2013 at 10:32 AM, Kirill Yukhin kirill.yuk...@gmail.com
wrote:
Bootstrap passing. Updated tests passing on
On Thu, 17 Apr 2014, Marc Glisse wrote:
On Thu, 17 Apr 2014, Richard Biener wrote:
This fixes PR60849 by properly rejecting non-boolean typed
comparisons from valid_gimple_rhs_p so they go through the
gimplification paths.
Could you also accept vector comparisons please?
Sure.
The patch below increases the number of coalescs we attempt
to also cover unary and binary operations. This improves
initial code generation for code like
int foo (int i, int j, int k, int l)
{
int res = i;
res += j;
res += k;
res += l;
return res;
}
from
;; res_3 = i_1(D) + j_2(D);
On Thu, 17 Apr 2014, Richard Biener wrote:
The patch below increases the number of coalescs we attempt
to also cover unary and binary operations. This improves
initial code generation for code like
int foo (int i, int j, int k, int l)
{
int res = i;
res += j;
res += k;
res
Hello,
My apologies for inconvenience.
Removed every appearance of -ftree-bitfield-merge from the patch and fixed an
issue with unions.
The rest of the patch is the same as before.
Regards,
Zoran Jovanovic
--
From: Yury Gribov y.gri...@samsung.com
Sent: Tuesday, March 25, 2014 11:57AM
To: Jakub Jelinek ja...@redhat.com, Eric Botcazou
ebotca...@adacore.com, gcc-patches@gcc.gnu.org, Hans-Peter Nilsson
h...@bitrange.com, rdsandif...@googlemail.com
On Wed, Apr 16, 2014 at 02:45:37PM -0400, DJ Delorie wrote:
I'll approve both patches, if you agree to think about a way to solve
this problem without module-specific configury changes for each such
command line option. I understand the usefulness of having
instrumentation, but the configure
On Wed, Apr 16, 2014 at 11:22:28AM +0200, Richard Biener wrote:
On Tue, 15 Apr 2014, Martin Jambor wrote:
Hi,
back in January in
http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00848.html Eric pointed
out a testcase where the problem was SRA not scalarizing an aggregate
because it was
On Thu, Apr 17, 2014 at 2:21 PM, Martin Jambor mjam...@suse.cz wrote:
On Wed, Apr 16, 2014 at 11:22:28AM +0200, Richard Biener wrote:
On Tue, 15 Apr 2014, Martin Jambor wrote:
Hi,
back in January in
http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00848.html Eric pointed
out a testcase
On Thu, Apr 17, 2014 at 10:53:07AM +0200, Richard Biener wrote:
On Thu, Apr 17, 2014 at 10:37 AM, tsaund...@mozilla.com wrote:
From: Trevor Saunders tsaund...@mozilla.com
Hi,
just removing some dead code.
bootstrapped + regtested against r209414 on x86_64-unknown-linux-gnu, ok?
Hi All,
We implemented enhancement for if-convert phase to recognize the
simplest conditional reduction and to transform it vectorizable form,
e.g. statement
if (A[i] != 0) num+= 1; will be recognized.
A new test-case is also provided.
Bootstrapping and regression testing did not show any
On Wed, Apr 16, 2014 at 10:39 PM, Jeff Law l...@redhat.com wrote:
On 03/26/14 17:44, Teresa Johnson wrote:
Recently I discovered that the profile updates being performed by jump
threading were incorrect in many cases, particularly in the case where
the threading path contains a joiner. Some
Hi,
This fixes infinite backtrace in __cxa_end_cleanup().
Regtest was finished with no regressions on arm-linux-gnueabi(sf).
The patch posted at:
http://gcc.gnu.org/ml/gcc-patches/2014-04/msg00496.html
Thanks in advance.
Best regards,
Merzlyakov Alexey
Hello,
He is a new version of the patch. It hookizes the mode-setting and
mode-toggling macros. Split in 2 parts.
Successfully bootstrapped/regtested on ix86 and SH4/SH4a.
I was able to do a limited build on Epiphany, if someone could give it a
try on it that would be great.
comments ?
Hi,
On Thu, 17 Apr 2014, Richard Biener wrote:
The patch below increases the number of coalescs we attempt
to also cover unary and binary operations.
This is not usually a good idea if not mitigated by things like register
pressure measurement and using target properties to determine if it's
and the toggle-support hookized
many thanks,
Christian
2014-04-02 Christian Bruel christian.br...@st.com
* target.def (mode_switching): New hook vector.
(toggle_init, toggle_destroy, toggle_set, toggle_test):
New mode toggle hooks.
* targhooks.h (default_toggle_test): Declare.
*
Respun. First two patches are for gcc, the last one is for upstream
LLVM.
The gcc part was bootstrapped and regtested on x86_64-unknown-linux-gnu
without regressions and bootstrapped on x86_64-unknown-linux-uclibc to
verify that the configury works as expected and that the library links
without
libsanitizer/ChangeLog
2014-04-02 Bernhard Reutner-Fischer al...@gcc.gnu.org
* configure.ac: Check for sizeof(struct statfs64).
* configure, config.h.in: Regenerate.
* sanitizer_common/sanitizer_platform_interceptors.h
(SANITIZER_INTERCEPT_STATFS64): Make
Conditionalize usage of dlvsym(), nanosleep(), usleep();
Conditionalize layout of struct sigaction and type of it's member
sa_flags.
Conditionalize glob_t members gl_closedir, gl_readdir, gl_opendir,
gl_flags, gl_lstat, gl_stat.
Check for availability of glob.h for use with above members.
Check
Conditionalize usage of dlvsym(), nanosleep(), usleep();
Conditionalize layout of struct sigaction and type of it's member
sa_flags.
Conditionalize glob_t members gl_closedir, gl_readdir, gl_opendir,
gl_flags, gl_lstat, gl_stat.
Check for availability of glob.h for use with above members.
Check
Hi,
If you are trying to modify the libsanitizer files, please read here:
https://code.google.com/p/address-sanitizer/wiki/HowToContribute
--kcc
On Thu, Apr 17, 2014 at 5:49 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
Conditionalize usage of dlvsym(), nanosleep(), usleep();
On 17 April 2014 16:07, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
Hi,
If you are trying to modify the libsanitizer files, please read here:
https://code.google.com/p/address-sanitizer/wiki/HowToContribute
I read that, thanks. Patch 3/3 is for current compiler-rt git repo,
Hi Bernd,
I have my copyright mark signed and the process has completed. Now I
am going to answer two more questions before my patch can be
commited right?
Did you copy any
files or text written by someone else in these changes?”
no
[Which files have you changed so
On 04/17/2014 02:00 AM, Tristan Gingold wrote:
On 16 Apr 2014, at 17:36, Richard Henderson r...@redhat.com wrote:
On 04/16/2014 12:39 AM, Eric Botcazou wrote:
The primary bit of rfc here is the hunk that applies to ada/types.h
with respect to Fat_Pointer. Given that the Ada type, as
On Thu, Apr 17, 2014 at 6:27 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On 17 April 2014 16:07, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
Hi,
If you are trying to modify the libsanitizer files, please read here:
On 2014-04-16, 3:19 PM, Tom de Vries wrote:
Vladimir,
All patches for the fuse-caller-save optimization have been ok-ed. The only
part
not approved is the MIPS-specific part.
The objection of Richard S. is not so much the patch itself, but more the idea
of the hook
Hi,
I checked in this preapproved patch to generate xor reg, reg when
optimizing for size.
H.J.
Index: ChangeLog
===
--- ChangeLog (revision 209487)
+++ ChangeLog (working copy)
@@ -1,3 +1,10 @@
+2014-04-17 H.J. Lu
Vladimir Makarov vmaka...@redhat.com writes:
On 2014-04-16, 3:19 PM, Tom de Vries wrote:
Vladimir,
All patches for the fuse-caller-save optimization have been ok-ed. The
only part
not approved is the MIPS-specific part.
The objection of Richard S. is not so much the patch itself, but
On 17 Apr 2014, at 16:50, Richard Henderson r...@redhat.com wrote:
On 04/17/2014 02:00 AM, Tristan Gingold wrote:
On 16 Apr 2014, at 17:36, Richard Henderson r...@redhat.com wrote:
On 04/16/2014 12:39 AM, Eric Botcazou wrote:
The primary bit of rfc here is the hunk that applies to
Ah hah.
/* Make sure we can put it into a register. */
if (STRICT_ALIGNMENT)
TYPE_ALIGN (record_type) = MIN (BIGGEST_ALIGNMENT, 2 * POINTER_SIZE);
AArch64 is not a STRICT_ALIGNMENT target, thus the mismatch.
I see. Initially this alignment promotion had been universal, but
On Wed, Apr 16, 2014 at 01:40:22PM -0400, Jason Merrill wrote:
On 04/15/2014 03:56 PM, Marek Polacek wrote:
The testsuite doesn't hit this code with C++, but does hit this code
with C. The thing is, if we have e.g.
enum { A = 128 };
void *fn1 (void) __attribute__((assume_aligned (A)));
then
+
+ /* At this stage we know that majority of GGC memory is reachable.
+ Growing the limits prevents unnecesary invocation of GGC. */
+ ggc_grow ();
ggc_collect ();
Isn't the collect here pointless? I see not in ENABLE_CHECKING, but
shouldn't this be abstracted
Hello,
Unfortunately, optimization is limited only to bit-fields that have same
bit-field representative (DECL_BIT_FIELD_REPRESENTATIVE), and fields from
different classes do have different representatives.
In given example optimization would merge accesses to x and y bit-fields from
Base
Hi,
we can fix this crash during error recovery very easily, by grouping
together the two conditions leading to skip early return, in complete
analogy with the single-declaration case (note that we explicitly commit
to tentative parse at the beginning of the function, thus we are good).
Hi all,
While looking at the build logs I noticed a warning while building
tree-ssa-loop-ivcanon.c about a potential use of an uninitialised variable.
This patchlet fixes that warning by initialising it to 0.
Tested arm-none-eabi.
Ok for trunk?
2014-04-17 Kyrylo Tkachov
On 17 April 2014 16:51:23 Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
On Thu, Apr 17, 2014 at 6:27 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On 17 April 2014 16:07, Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
Hi,
If you are trying to
On 2014-04-17, 11:29 AM, Richard Sandiford wrote:
Vladimir Makarov vmaka...@redhat.com writes:
On 2014-04-16, 3:19 PM, Tom de Vries wrote:
Vladimir,
All patches for the fuse-caller-save optimization have been ok-ed. The
only part
not approved is the MIPS-specific part.
The objection of
Hi all,
While configuring libgfortran I'm getting this message:
libgfortran/configure: line 25938: test: =: unary operator expected
The script doesn't fail and continues afterwards, but I don't think it's
supposed to give that warning.
This patch makes it go away and makes it more consistent
On Thu, Apr 17, 2014 at 8:45 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On 17 April 2014 16:51:23 Konstantin Serebryany
konstantin.s.serebry...@gmail.com wrote:
On Thu, Apr 17, 2014 at 6:27 PM, Bernhard Reutner-Fischer
rep.dot@gmail.com wrote:
On 17 April 2014 16:07,
Hello!
I am not against it..
However I think there is no danger. I see no potential use of
uninitialized variable.
The use of n_unroll is guarded by n_unroll_found.
Best regards,
Daniel Marjamäki
Hi,
GET_MODE returns VOIDmode on CONST_INT. It happens with -O0. This
patch uses counter_mode on count_exp to get mode. Tested on
Linux/x86-64 without regressions. OK for trunk and 4.9 branch?
Thanks.
H.J.
---
gcc/
2014-04-17 H.J. Lu hongjiu...@intel.com
PR target/60868
On April 17, 2014 6:03:13 PM CEST, Jan Hubicka hubi...@ucw.cz wrote:
+
+ /* At this stage we know that majority of GGC memory is
reachable.
+ Growing the limits prevents unnecesary invocation of GGC.
*/
+ ggc_grow ();
ggc_collect ();
Isn't the collect here
On April 17, 2014 6:03:13 PM CEST, Jan Hubicka hubi...@ucw.cz wrote:
+
+ /* At this stage we know that majority of GGC memory is
reachable.
+ Growing the limits prevents unnecesary invocation of GGC.
*/
+ ggc_grow ();
ggc_collect ();
Isn't the collect here
On 04/17/2014 08:35 AM, Tristan Gingold wrote:
What about this compile-time check:
package Fatptralign is
type String_Acc is access String;
type Integer_acc is access Integer;
pragma Compile_Time_Error
(String_Acc'Alignment = 1 * Integer_Acc'Alignment,
Fat pointer are
Ping.
On Wed, Feb 5, 2014 at 4:31 PM, Sriraman Tallam tmsri...@google.com wrote:
Hi,
I would like this patch reviewed and considered for commit when
Stage 1 is active again.
Patch Description:
A C++ thunk's section name is set to be the same as the original function's
section name for
Hi,
I think for 4.10 we should revisit inliner behaviour to be more LTO and LTO+FDO
ready. This is first of small patches I made to sanitize behaviour of current
bounds.
The main problem LTO brings is that we get way too many inline candidates. In
per-file
model one gets only small percentage
On April 17, 2014 7:18:05 PM CEST, Jan Hubicka hubi...@ucw.cz wrote:
On April 17, 2014 6:03:13 PM CEST, Jan Hubicka hubi...@ucw.cz
wrote:
+
+ /* At this stage we know that majority of GGC memory is
reachable.
+ Growing the limits prevents unnecesary invocation of
GGC.
*/
On 27 Mar 17:16, Jakub Jelinek wrote:
On Thu, Mar 27, 2014 at 08:13:00PM +0400, Ilya Verbin wrote:
On 27 Mar 15:02, Jakub Jelinek wrote:
The tables need to be created before IPA, that way it really shouldn't
matter in what order you emit them. E.g. the outlined target functions
could
On Thu, 2014-04-17 at 19:52 +0200, Jan Hubicka wrote:
[...]
Index: doc/invoke.texi
===
--- doc/invoke.texi (revision 209461)
+++ doc/invoke.texi (working copy)
@@ -9409,7 +9409,8 @@ before applying @option{--param inline-u
This patch from Chris Manghane marks various expression types as
immutable: numerics, constants, type info, address of, type conversion
when appropriate. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 194e0f47c9e5 go/expressions.cc
---
This looks fine. LIPO has similar change too. Other directions worth
looking into:
1) To model icache effect better, weighted callee size need to be
used with profile. The weight for BB may look like: min(1,
FREQ(BB)/FREQ(ENTRY)).
2) When function splitting is turned on, are any inline
Hi Lin,
On Thu, 17 Apr 2014 22:29:14, Lin Zuojian wrote:
Hi Bernd,
I have my copyright mark signed and the process has completed. Now I
am going to answer two more questions before my patch can be
commited right?
Did you copy any
files or text written by someone else in these changes?”
This patch to the Go frontend fixes it to not convert the function type
in a call when calling an interface method. The function type of an
interface method is not correct, since it does not include the receiver,
but the type of the method field is correct, and as such should not be
converted.
On 15/04/14 22:58, DJ Delorie wrote:
I typically leave the mode off when the operand accepts a CONST_INT as
I've had problems with patterns matching CONST_INTs otherwise, as
CONST_INT rtx's do not have a mode (or have VOIDmode).
(yes, I know gcc is supposed to accomodate that, but like I said,
On Tue, 15 Apr 2014, DJ Delorie wrote:
I wasn't sure what to do with that array, since it was static and
couldn't have empty slots in them like the arrays in tree.h. Also,
do we need to have *every* type in that list? What's the rule for
whether a type gets installed there or not? The
On 04/17/2014 08:56 AM, Eric Botcazou wrote:
I presume that the attached kludge is sufficient to make it work?
* fe.h (Compiler_Abort): Replace Fat_Pointer by String.
(Error_Msg_N): Likewise.
(Error_Msg_NE): Likewise.
(Get_External_Name_With_Suffix): Likewise.
This patch from Chris Manghane changes the Go frontend to use the
backend interface for global constants. Bootstrapped and ran Go
testsuite on x86_64-unknown-linux-gnu. Committed to mainline.
Ian
2014-04-17 Chris Manghane cm...@google.com
* go-gcc.cc
On Wed, Apr 16, 2014 at 11:35 PM, Jeff Law l...@redhat.com wrote:
I'd like to ping 2 patches:
http://gcc.gnu.org/ml/gcc-patches/2014-01/msg00140.html
- Ensure GET_MODE_{SIZE,INNER,NUNITS} (const) is constant rather than
memory load after optimization (I'd like to keep the current
I committed the following patch as obvious to fix the PowerPC build issue that
came up with changes to machmode.h. These changes allow the compiler to build
and bootstrap. Submitted as subversion id 209498.
2014-04-17 Michael Meissner meiss...@linux.vnet.ibm.com
PR target/60876
It seems a little inconsistent, however, that *movqi_real and
*xorqi3_real don't specify modes but *movhi_real and
*andqi_real/*iorqi_real do (and they also accept CONST_INTs). Not that
I'm advocating generating more warnings, but my inner OCD likes
consistency :)
Adding the mode might
This patch from Peter Collingbourne avoids an unnecessary gccgo
extension in libgo. Bootstrapped and ran Go testsuite on
x86_64-unknown-linux-gnu. Committed to mainline.
Ian
diff -r 801009f33610 libgo/go/syscall/libcall_posix.go
--- a/libgo/go/syscall/libcall_posix.go Thu Apr 17 16:01:58 2014
This patch from Peter Collingbourne changes libgo to use the builtin
delete function rather than the old map deletion syntax. Bootstrapped
and ran Go testsuite on x86_64-unknown-linux-gnu. Committed to
mainline.
Ian
diff -r 1e27a38c43ea libgo/go/syscall/syscall_unix.go
---
This patch from Peter Collingbourne avoids duplicate function
declarations in the generated libcalls.go file when building the syscall
package. Bootstrapped and ran Go testsuite on x86_64-unknown-linux-gnu.
Committed to mainline.
Ian
diff -r 5009262c3e56 libgo/go/syscall/mksyscall.awk
---
Hi,
I missed a case in the vector API work for little endian. When VSX is
enabled, the vec_mergeh and vec_mergel interfaces for 4x32 vectors are
translated into xxmrghw and xxmrglw. The patterns for these were not
adjusted for little endian. This patch fixes this and adds tests for
V4SI and
Hi,
Here is the patch after the Jakub's review, and Jakub helps with the
coding style.
--
* asan.c (asan_emit_stack_protection):
Force the base to align to appropriate bits if STRICT_ALIGNMENT. Set
shadow_mem align to appropriate bits if STRICT_ALIGNMENT.
* cfgexpand.c
Hi Bernd,
a) On which target(s) did you boot-strap your patch?
I just run it on x86, can't run it on ARM, because Android is not a
posix system, nor a System V compatible system. And my code does not
effect x86.
b) Did you run the testsuite?
Yes, but again my code does not effect
Resulting from discussion here:
http://gcc.gnu.org/ml/gcc/2014-04/msg00125.html
ChangeLog:
* g++.dg/cpp0x/nsdmi-union5.C: Change to runtime test.
Index: gcc/testsuite/g++.dg/cpp0x/nsdmi-union5.C
===
---
This patch implements a new warning that warns when controlling
expression of a switch has boolean value. (Intentionally I don't
warn if the controlling expression is (un)signed:1 bit-field.)
I guess the question is if this should be enabled by default or
deserves some new warning option. Since
On Fri, 18 Apr 2014, Marek Polacek wrote:
This patch implements a new warning that warns when controlling
expression of a switch has boolean value. (Intentionally I don't
warn if the controlling expression is (un)signed:1 bit-field.)
I guess the question is if this should be enabled by default
91 matches
Mail list logo