On Mon, Nov 12, 2012 at 6:34 AM, Gopalasubramanian, Ganesh
ganesh.gopalasubraman...@amd.com wrote:
You can see from the changes of sse.md that this is functionally a no-op
change.
Sseshuf replaces sselog.
So, do you mean it should be added with sselog instead of sseadd?
Adding it with sseadd
This is a target-specific blockage insn, but with the general form
found in all targets defining it. The default blockage is an empty
asm-volatile, which is what cse_insn recognized. The blockage insn is
there to prevent scheduling of the critical insns and register
values. It's almost
On Tue, Nov 6, 2012 at 9:12 AM, Eric Botcazou ebotca...@adacore.com wrote:
2012-11-05 Uros Bizjak ubiz...@gmail.com
Kaz Kojima kkoj...@gcc.gnu.org
PR target/41993
* mode-switching.c (create_pre_exit): Set return_copy to
last_insn when copy_start is a pseudo
I strongly doubt that they will be different from the options
supported both in cc and fbe in the Solaris Studio 12.3 release:
They need to provide some form of backward compatibility though, they cannot
break the interface of 'as' like that. Apparently 'fbe' has had its own set
of -xarch
On Sat, Nov 10, 2012 at 07:54:34PM +0100, Tobias Burnus wrote:
2012-11-10 Tobias Burnus bur...@net-b.de
Jakub Jelinek ja...@redhat.com
* asan.c (maybe_instrument_builtin_call): Set *iter
to gsi for the call at the end.
(transform_statements): Leave loop
It looks to me, that we in fact want:
--cut here--
Index: mode-switching.c
===
--- mode-switching.c(revision 193407)
+++ mode-switching.c(working copy)
@@ -330,7 +330,7 @@
short_block = 1;
On Fri, Nov 09, 2012 at 12:03:09PM -0800, Kostya Serebryany wrote:
Why depending on gtest is bad?
Because gcc already has way too many dependencies (starting from gmp, mpfr,
libmpc, graphite dependencies, java depenencies), adding another one just
for the sake of a couple of asan tests would
On Mon, Nov 12, 2012 at 9:51 AM, Vladimir Yakovlev vbyakov...@gmail.com wrote:
We have not do any changes in mode-switching.c if we use
pass_mode_switching.pass.execute () for call of the
optimize_mode_switching() in i386.c.
Even better ;)
In addition to this patch, there are two remaining
Hello,
In this PR, a set to a global reg is removed by fast-dce and ud-dce.
Such sets should always be preserved.
The proposed fix is to make global regs always live. This is done in
the df-* parts of the patch from Jakub (the test case is also his).
In addition, ud-dce should consider sets of
On 06/11/12 18:49, Ian Bolton wrote:
OK for trunk?
Cheers,
Ian
2012-11-06 Ian Bolton ian.bol...@arm.com
* gcc/config/aarch64/aarch64.md (*compare_negmode): New pattern.
* gcc/testsuite/gcc.target/aarch64/cmn.c: New test.
* gcc/testsuite/gcc.target/aarch64/adds.c: New
On 06/11/12 18:56, Ian Bolton wrote:
Where a CSEL can return the value 1 as one of the alternatives,
it is usually more efficient to use a CSINC than a CSEL (and
never less efficient), since the value of 1 can be derived from
wzr, rather than needing to set it up in a register first.
This patch
On Fri, 2012-11-09 at 21:54 +0100, Andi Kleen wrote:
I'm not sure this is quite true. If a libitm-executed transaction is
It's just a convention. You don't have to use it.
That's true ...
Not doing it will
just make abort profiling harder.
... but I disagree with this one. This won't
An minor obvious fix, found by Coverity.
Committed as Rev. 193428 after all-language-bootstrap/regtesting on
x86-64-gnu-linux.
Tobias
Index: gcc/ChangeLog
===
--- gcc/ChangeLog (Revision 193427)
+++ gcc/ChangeLog (Arbeitskopie)
Thanks for the report Damian! And sorry for the breakage.
Fixed with the following obvious patch (committed as Rev. 193429) after
an all-language bootstrap/regtesting
Tobias
Index: gcc/fortran/ChangeLog
===
---
[I am replying to several emails at once as I believe they are logically
connected]
Tobias Burnus bur...@net-b.de writes:
I tried the 01/10 to 10/10 patch series but it doesn't trigger for the
following test case:
#include stdlib.h
#include stdio.h
int
main() {
int *i;
i =
On 2012-11-02 15:57 , Dodji Seketeli wrote:
/* AddressSanitizer, a fast memory error detector.
- Copyright (C) 2011 Free Software Foundation, Inc.
+ Copyright (C) 2011, 2012 Free Software Foundation, Inc.
I *think* we should only mention 2012, but I don't know if code in
Tobias Burnus bur...@net-b.de writes:
* -fno-address-sanitizer doesn't work (it does in Clang); it is
explicitly disabled via RejectNegative in gcc/common.opt
Fixed in common.opt by removing the RejectNegative.
I am thus sending the updated patch.
* common.opt: Add
Diego Novillo dnovi...@google.com writes:
I believe they layout the stack from right to left (top is to the
right). Feels like reading a middle earth map. Kostya, is my
recollection correct?
Yes, Konstantin replied to this already but I forgot to update the
patch cover letter (that I keep
Diego Novillo dnovi...@google.com writes:
On 2012-11-02 16:01 , Dodji Seketeli wrote:
* varasm.c: Include asan.h.
(assemble_noswitch_variable): Grow size by asan_red_zone_size
if decl is asan protected.
(place_block_symbol): Likewise.
(assemble_variable): If decl
Diego Novillo dnovi...@google.com writes:
On 2012-11-02 16:05 , Dodji Seketeli wrote:
+static bool
+maybe_instrument_builtin_call (gimple_stmt_iterator *iter)
+{
+ gimple call = gsi_stmt (*iter);
+ location_t loc = gimple_location (call);
+
+ if (!is_gimple_call (call))
+return
Diego Novillo dnovi...@google.com writes:
On 2012-11-02 16:10 , Dodji Seketeli wrote:
* configure.ac: Add libsanitizer to target_libraries.
* Makefile.def: Ditto.
* configure: Regenerate.
* Makefile.in: Regenerate.
* libsanitizer: New directory for asan
On Mon, Nov 12, 2012 at 12:30:37PM +0100, Dodji Seketeli wrote:
+ For this function, the stack protected by asan will be organized as
+ follows, from the top of the stack to the bottom:
+
+ Slot 1/ [red zone of 32 bytes called 'RIGHT RedZone']
+
+ Slot 2/ [24 bytes for variable 'a']
+
+
Tobias Burnus bur...@net-b.de writes:
The attached test case ICEs (segfault) both on the asan branch and on
the trunk with Dodji's patches:
Thank you for reporting this.
I believe the neqw series of patches I have juste posted address this
issue.
To ease the testing, you can check out an
Hi,
This patch is a cleanup of the initialisation code for the
SIMD builtins in the AArch64 port.
We do this in preparation for later patches supporting
TARGET_BUILTIN_DECL and
TARGET_VECTORIZE_BUILTIN_VECTORIZABLE_FUNCTION.
This patch supersedes another, similar patch I posted a few weeks
ago
Hello,
this patch lets us fold a bit_field_ref of a constructor even when the
elements are vectors. Writing a testcase is not that convenient because of
the lack of a dead code elimination pass after forwprop4 (RTL doesn't
always remove everything either), but I see a clear difference on this
On 11/12/2012 01:50 AM, Marc Glisse wrote:
When we are changing the whole type, I assume the point of using this
attribute instead of the standard solutions (move it to some other
(inline) namespace, for instance) is the -Wabi-tag warning?
Right.
Jason
http://gcc.gnu.org/ml/gcc-patches/2012-10/msg01521.html
Ping.
Bye,
Ulrich
--
Dr. Ulrich Weigand
GNU Toolchain for Linux on System z and Cell BE
ulrich.weig...@de.ibm.com
Eric Botcazou ebotca...@adacore.com writes:
I strongly doubt that they will be different from the options
supported both in cc and fbe in the Solaris Studio 12.3 release:
They need to provide some form of backward compatibility though, they cannot
break the interface of 'as' like that.
On Tue, 21 Aug 2012, Tobias Burnus wrote:
I noticed you are using q.../q, as in qcodee/code/q,
which we usually don't. Why that?
My impression was that a one-letter code didn't stand out enough and
looked rather odd; if you think it improves consistency or readability,
feel free to change
Gerald Pfeifer wrote:
One question: in general we are using the term command-line option.
You are introducing two uses of flag, and the two existing uses of
that are also in the Fortran section. Is this Fortran terminology or
would it be fine to make this more consistent?
Well, flag is GCC
No, quite the contrary. as is just a (sometimes partial) backport of
Studio fbe, though it's hard to tell exactly which Studio version of fbe
forms the basis of as. Especially for the Solaris 10 as patches, only
particular bugfixes/enhancements have been backported.
Backward compatibility
Dodji Seketeli wrote:
I believe the neqw series of patches I have juste posted address this
issue.
Thanks a lot for your merging work! I am really looking forward to have
it available on the trunk!
I guess that your updated patch fixes the issue as it incorporates
Jakub's patch. (I have
Eric Botcazou ebotca...@adacore.com writes:
No, quite the contrary. as is just a (sometimes partial) backport of
Studio fbe, though it's hard to tell exactly which Studio version of fbe
forms the basis of as. Especially for the Solaris 10 as patches, only
particular bugfixes/enhancements
Following a request from Jakub, and given the fact that the patch set
have been reviewed by Diego, I have committed the last set of patches I
have posted to trunk.
This will hopefully ease the polishing work that has started already.
I am of course watching for the fall-outs.
--
Jakub Jelinek ja...@redhat.com writes:
On Mon, Nov 12, 2012 at 12:30:37PM +0100, Dodji Seketeli wrote:
+ For this function, the stack protected by asan will be organized as
+ follows, from the top of the stack to the bottom:
+
+ Slot 1/ [red zone of 32 bytes called 'RIGHT RedZone']
+
+
On Mon, Nov 12, 2012 at 05:07:42PM +0100, Dodji Seketeli wrote:
Following a request from Jakub, and given the fact that the patch set
have been reviewed by Diego, I have committed the last set of patches I
have posted to trunk.
Thanks, I've committed as obvious the following formatting
On Mon, Nov 12, 2012 at 04:45:55PM +0100, Tobias Burnus wrote:
First, I have a small hyphen fix patch, which is on top of your
merge branch. (The asan branch itself is okay.)
This patch is preapproved with appropriate ChangeLog entry.
Thanks.
--- invoke.texi.orig2012-11-12
Jakub Jelinek:
On Mon, Nov 12, 2012 at 05:07:42PM +0100, Dodji Seketeli wrote:
Following a request from Jakub, and given the fact that the patch set
have been reviewed by Diego, I have committed the last set of patches I
have posted to trunk.
Thanks, I've committed as obvious the following
Jakub Jelinek ja...@redhat.com writes:
The bug is elsewhere, the following patch should fix this
(and I've reordered the assignments according to the call arg
number, so that it is more readable at the same time).
Ok for trunk?
2012-11-12 Jakub Jelinek ja...@redhat.com
* asan.c
Folks, please remember that the Clang flag has recently changed to
-fsanitize=address (-fsanitize=thread).
This is hopefully the last syntax change there.
--kcc
On Mon, Nov 12, 2012 at 8:45 AM, Tobias Burnus bur...@net-b.de wrote:
Jakub Jelinek:
On Mon, Nov 12, 2012 at 05:07:42PM +0100, Dodji
Another ICE:
% cat test.ii
int i;
% g++ -faddress-sanitizer -c -g -O1 test.ii
test.ii:1:7: internal compiler error: Segmentation fault
int i;
^
0xa5cb5f crash_signal
/home/markus/gcc/gcc/toplev.c:334
inconsistent DWARF line number info
0x4cf588 cp_classify_record
On Mon, Nov 12, 2012 at 12:45 AM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Nov 09, 2012 at 12:03:09PM -0800, Kostya Serebryany wrote:
Why depending on gtest is bad?
Because gcc already has way too many dependencies (starting from gmp, mpfr,
libmpc, graphite dependencies, java
/libstdc++-v3/include
-I/sw/src/fink.build/gcc48-4.8.0-1000/gcc-4.8-20121112/libstdc++-v3/libsupc++
-I/sw/src/fink.build/gcc48-4.8.0-1000/gcc-4.8-20121112/libstdc++-v3/include/backward
-I/sw/src/fink.build/gcc48-4.8.0-1000/gcc-4.8-20121112/libstdc++-v3/testsuite/util
-L/sw/src/fink.build/gcc48
On Nov 9, 2012, at 1:42 AM, Eric Botcazou ebotca...@adacore.com wrote:
this is a regression present on the mainline and 4.7 branch. -fvar-tracking
is automatically enabled on Darwin when -g is passed, including at -O0. This
results in bloated and incomplete debug info.
OK for mainline and
On Fri, Nov 9, 2012 at 11:13 AM, Jakub Jelinek ja...@redhat.com wrote:
On Fri, Nov 09, 2012 at 11:05:37AM -0800, Wei Mi wrote:
gtest integrate multiple tests into the same file with each test being
a single line check. I cannot think out a method to migrate it to
dejagnu without using gtest,
-darwin12.2.0
-I/sw/src/fink.build/gcc48-4.8.0-1000/darwin_objdir/x86_64-apple-darwin12.2.0/libstdc++-v3/include
-I/sw/src/fink.build/gcc48-4.8.0-1000/gcc-4.8-20121112/libstdc++-v3/libsupc++
-I/sw/src/fink.build/gcc48-4.8.0-1000/gcc-4.8-20121112/libstdc++-v3/include/backward
-I/sw/src/fink.build
Jack Howarth wrote:
Copying over the lib/interception/mach_override directory from
llvm.org's compiler-rt 3.2 branch into libsanitizer/interception
allows the build of libsanitizer to proceed. I noticed that the
mach_override subdirectory has a license file which shows...
I was about to
On Nov 12, 2012, at 9:32 AM, Wei Mi w...@google.com wrote:
Using setjmp/longjmp to do multiple tests in a single testfile, the
test statements in the front could affect the tests in the back. gtest
will fork a new process for every test statement.
Some targets don't have fork. :-( Ah.
Hello Jack,
Jack Howarth howa...@bromo.med.uc.edu writes:
Dodji,
I am finding that at r193442 bootstrapping on
x86_64-apple-darwin12 fails with...
../../../../gcc-4.8-20121112/libsanitizer/interception/interception_mac.cc:16:41:
fatal error: mach_override/mach_override.h
On 10/22/2012 08:39 PM, David Miller wrote:
+ /* Compare and Branch is limited to +-2KB. If it is too far away,
+ change
+
+ cxbne X, Y, .LC30
+
+ to
+
+ cxbe X, Y, .+12
+ ba,pt xcc, .LC30
+ nop */
Based on your no-control-after cbcond comment at the top
of
On Mon, Nov 12, 2012 at 06:37:06PM +0100, Tobias Burnus wrote:
Jack Howarth wrote:
Copying over the lib/interception/mach_override directory from
llvm.org's compiler-rt 3.2 branch into libsanitizer/interception
allows the build of libsanitizer to proceed. I noticed that the
On Mon, Nov 12, 2012 at 06:13:08PM +0100, Markus Trippelsdorf wrote:
Another ICE:
% cat test.ii
int i;
% g++ -faddress-sanitizer -c -g -O1 test.ii
test.ii:1:7: internal compiler error: Segmentation fault
int i;
^
The RECORD_TYPE doesn't have lang specific payload allocated
On Mon, Nov 12, 2012 at 09:32:04AM -0800, Wei Mi wrote:
Using setjmp/longjmp to do multiple tests in a single testfile, the
test statements in the front could affect the tests in the back. gtest
will fork a new process for every test statement. The forked process
will do only one test and skip
On Mon, Nov 12, 2012 at 10:05 AM, Jakub Jelinek ja...@redhat.com wrote:
On Mon, Nov 12, 2012 at 09:32:04AM -0800, Wei Mi wrote:
Using setjmp/longjmp to do multiple tests in a single testfile, the
test statements in the front could affect the tests in the back. gtest
will fork a new process for
AIX has added its equivalent to PPC64 Linux Large TOC support. The
appended patch implements GCC support and enables it if the AIX system
tools support the new relocs. This is not the default for XLC and
will not be the default for GCC.
GCC testsuite produces same results if run with
Hi,
I don't insist that we use gtest for gcc-asan, I just say that this is
the simplest approach to get 2.5K test suite into gcc,
and the only approach my team will be able to maintain.
gtest is not as portable as the rest of gcc, but neither is asan
run-time library (which is more
For TODO_update_ssa, when we insert tsan_write(a), current function's
ssa_renaming_needed flag will be set in finalize_ssa_defs because we
insert non-ssaname vdef. An assertion in execute_todo will check
whether we have TODO_update_ssa set when current function's
ssa_renaming_needed flag is set.
On Mon, Nov 12, 2012 at 3:47 AM, Dodji Seketeli do...@seketeli.org wrote:
Diego Novillo dnovi...@google.com writes:
On 2012-11-02 16:10 , Dodji Seketeli wrote:
* configure.ac: Add libsanitizer to target_libraries.
* Makefile.def: Ditto.
* configure: Regenerate.
*
Used HJ's commit as motivation to also add the generated files of
libquadmath files to the gcc_update file.
Committed as .
Tobias
Index: contrib/ChangeLog
===
--- contrib/ChangeLog (Revision 193447)
+++ contrib/ChangeLog
One minor difference between the gc and gccgo libraries is that the
error messages returned by the gc library force the first letter of the
error message to be lower case. This normally doesn't matter, but of
course it does matter for code that compares error strings directly.
Such code is of
From: Rainer Orth r...@cebitec.uni-bielefeld.de
Date: Mon, 12 Nov 2012 16:46:37 +0100
Eric Botcazou ebotca...@adacore.com writes:
No, quite the contrary. as is just a (sometimes partial) backport of
Studio fbe, though it's hard to tell exactly which Studio version of fbe
forms the basis of
From: Eric Botcazou ebotca...@adacore.com
Date: Mon, 12 Nov 2012 09:35:48 +0100
I strongly doubt that they will be different from the options
supported both in cc and fbe in the Solaris Studio 12.3 release:
They need to provide some form of backward compatibility though, they cannot
break
From: Richard Henderson r...@redhat.com
Date: Mon, 12 Nov 2012 09:56:21 -0800
On 10/22/2012 08:39 PM, David Miller wrote:
+ /* Compare and Branch is limited to +-2KB. If it is too far away,
+ change
+
+ cxbne X, Y, .LC30
+
+ to
+
+ cxbe X, Y, .+12
+ ba,pt xcc,
Hi!
This patch fixes two ICEs on the testcases, one is ICE in
expand_vector_condition which wouldn't handle VECTOR_CST as first argument
of VEC_COND_EXPR (plus a change to make sure it is folded earlier if
possible), and the vectorizable_condition change makes sure the condition
is actually using
On 11/12/2012 01:04 AM, Steven Bosscher wrote:
gcc/
PR rtl-optimization/51447
* df-scan.c (df_get_entry_block_def_set): Add global regs to the set.
* df-problems.c (df_lr_local_compute): Make global regs always live.
* dce.c (deletable_insn_p): Make insns setting a
Hi!
The following testcase was ICEing on powerpc64-linux.
While force_nonfallthru_and_redirect properly adjusted the LABEL_REFs
in asm goto from the old fallthru block to a new one, it didn't adjust
LABEL_NUSES and didn't adjust JUMP_LABEL resp. REG_LABEL_TARGET.
Fixed thusly,
On 11/12/2012 11:43 AM, Jakub Jelinek wrote:
2012-11-12 Jakub Jelinek ja...@redhat.com
PR tree-optimization/55281
* tree-vect-generic.c (expand_vector_condition): Accept any
is_gimple_val rather than just SSA_NAME if not COMPARISON_CLASS_P.
* fold-const.c
On 11/12/2012 11:48 AM, Jakub Jelinek wrote:
2012-11-12 Jakub Jelinek ja...@redhat.com
PR rtl-optimization/54127
* cfgrtl.c (force_nonfallthru_and_redirect): When redirecting
asm goto labels from BB_HEAD (e-dest) to target bb, decrement
LABEL_NUSES of BB_HEAD
On 2012.11.12 at 19:02 +0100, Jakub Jelinek wrote:
On Mon, Nov 12, 2012 at 06:13:08PM +0100, Markus Trippelsdorf wrote:
Another ICE:
% cat test.ii
int i;
% g++ -faddress-sanitizer -c -g -O1 test.ii
test.ii:1:7: internal compiler error: Segmentation fault
int i;
^
On Thu, Nov 1, 2012 at 12:52 PM, do...@redhat.com wrote:
From: Dodji Seketeli do...@seketeli.org
Hello,
The set of patches following this message represents the work that
happened on the asan branch to build up the Address Sanitizer work
started in the Google branch.
Address Sanitizer
On Wed, Nov 7, 2012 at 5:50 AM, Rainer Orth r...@cebitec.uni-bielefeld.de
wrote:
Gerald Pfeifer ger...@pfeifer.com writes:
Just a small note, in the following
+#ifdef __FreeBSD__
+# define DEFAULT_PROCESS_FILENAME /proc/curproc/file
+#elif defined(HAVE_GETEXECNAME)
+# define
On Mon, Nov 12, 2012 at 12:39 PM, H.J. Lu hjl.to...@gmail.com wrote:
Don't we need a bugzilla component for Sanitizer?
other?
Ian
std::atomic_future was removed by N3170
* include/std/future (atomic_future): Remove declaration of
non-standard name.
Tested x86_64-linux, committed to trunk, 4.7 to follow.
commit 27ccc75ad9c7955aa432f0dc35de94965b7fc3e6
Author: Jonathan Wakely jwakely@gmail.com
Date: Mon
They show up when a lattice value changed is to CONSTANT.
Tested on x86_64-suse-linux, applied on mainline as obvious.
2012-11-12 Eric Botcazou ebotca...@adacore.com
* tree-ssa-ccp.c (dump_lattice_value) CONSTANT: Fix duplication.
--
Eric BotcazouIndex: tree-ssa-ccp.c
Hi,
Linux/x32 doesn't have __NR_mmap2/__NR_fstat64 and uses
__NR_mmap/__NR_fstat, just like Linux/x86-64. Tested on Linux/x32.
OK to install?
Thanks.
H.J.
--
2012-11-12 H.J. Lu hongjiu...@intel.com
PR other/55292
* sanitizer_common/sanitizer_linux.cc (internal_mmap): Use
Hi,
I know we are in stage3, I thought I would send this out now for
review as I finally have it ready for consumption as I finally got
around to removing the limitation of it only working on big-endian.
This pass was originally written by Adam Nemet while he was at Cavium.
I modified it to
On Mon, Nov 12, 2012 at 4:50 PM, Andrew Pinski
andrew.pin...@caviumnetworks.com wrote:
Hi,
I know we are in stage3, I thought I would send this out now for
review as I finally have it ready for consumption as I finally got
around to removing the limitation of it only working on big-endian.
On Mon, 12 Nov 2012, Andrew Pinski wrote:
Also does gtest support cross testing; that is testing over rsh/ssh
and testing via a simulator? We should require that as a requirement
also when it comes to testing infrastructures.
Unfortunately the existing guality tests work by C sources built
The attached change fixes PR target/55195.
A recent change to shorten_branches broke branch shortening on PA. Several
years ago we changed to using function calls to estimate the length of the
call patterns because the cond expressions became unmanageable. The
functions returned the maximum
Hi Jason,
Made the changes. Also fixed one more segfault bug when ifunc is
not supported.
Thanks,
-Sri.
On Sun, Nov 11, 2012 at 9:04 PM, Jason Merrill ja...@redhat.com wrote:
On 11/09/2012 08:33 PM, Sriraman Tallam wrote:
+ /* Skip calling decls_match for versioned functions. */
While the fallout(*) from the libsanitizer commit is handled,
it's obvious it should have a noconfigdirs= section in
toplevel/configure.ac like the other target libs. Here's what I
committed after observing that a cris-elf build passed, where it
previously failed building libsanitizer which
On Mon, Nov 12, 2012 at 6:36 PM, H.J. Lu hjl.to...@gmail.com wrote:
Hi,
Linux/x32 doesn't have __NR_mmap2/__NR_fstat64 and uses
__NR_mmap/__NR_fstat, just like Linux/x86-64. Tested on Linux/x32.
OK to install?
Patches to libsanitizer should be sent upstream. We should only
contain a copy
On Mon, Nov 12, 2012 at 6:02 PM, Diego Novillo dnovi...@google.com wrote:
On Mon, Nov 12, 2012 at 6:36 PM, H.J. Lu hjl.to...@gmail.com wrote:
Hi,
Linux/x32 doesn't have __NR_mmap2/__NR_fstat64 and uses
__NR_mmap/__NR_fstat, just like Linux/x86-64. Tested on Linux/x32.
OK to install?
On 11/12/2012 08:11 PM, Sriraman Tallam wrote:
+!targetm.target_option.function_versions (fn,
+TREE_PURPOSE (match)))
The second argument should be lined up with the left paren if it's on a
different line. Perhaps formatting this as
From: Rainer Orth r...@cebitec.uni-bielefeld.de
Date: Fri, 26 Oct 2012 11:14:33 +0200
I tried a bootstrap on Solaris 11.1, but ran into lots of comparison
failures I've not yet investigated.
I started working on this patch again, in order to incorporate
Richard Henderson's feedback, and I am
On Mon, Nov 12, 2012 at 6:02 PM, Diego Novillo dnovi...@google.com wrote:
On Mon, Nov 12, 2012 at 6:36 PM, H.J. Lu hjl.to...@gmail.com wrote:
Hi,
Linux/x32 doesn't have __NR_mmap2/__NR_fstat64 and uses
__NR_mmap/__NR_fstat, just like Linux/x86-64. Tested on Linux/x32.
OK to install?
gcc/asan.c probably should have been split into two files because it
works at multiple levels. But given that it invokes
ASM_GENERATE_INTERNAL_LABEL, it needs to include tm_p.h to include
target-protos.h.
Committed as obvious to allow AIX bootstrap to proceed.
* asan.c: Include tm_p.h
-Original Message-
From: Joseph Myers [mailto:jos...@codesourcery.com]
Sent: Saturday, November 10, 2012 12:35 AM
To: Terry Guo
Cc: gcc-patches@gcc.gnu.org
Subject: RE: [RFC] New feature to reuse one multilib among different
targets
On Fri, 9 Nov 2012, Terry Guo wrote:
You
This has broken the build on every Linux target that hasn't added
the necessary cpu specific code to asan_linux.cc
I'm working on the sparc bits, but I'm really surprised this
happened.
...those of you who don't have the equivalent of the patch
below in your ports, that is. You'll likely only notice through
a slightly reduced debugging experience and
g++.dg/debug/dwarf2/non-virtual-thunk.C failing. Yes, the docs
should mention these functions need to be called IMHO obviously,
libsanitizer/
* asan/asan_linux.cc (GetPcSpBp): Add sparc support.
---
libsanitizer/ChangeLog.asan | 4
libsanitizer/asan/asan_linux.cc | 14 ++
2 files changed, 18 insertions(+)
diff --git a/libsanitizer/ChangeLog.asan b/libsanitizer/ChangeLog.asan
index
On Mon, Nov 12, 2012 at 6:59 PM, Andrew Pinski pins...@gmail.com wrote:
On Mon, Nov 12, 2012 at 6:02 PM, Diego Novillo dnovi...@google.com wrote:
On Mon, Nov 12, 2012 at 6:36 PM, H.J. Lu hjl.to...@gmail.com wrote:
Hi,
Linux/x32 doesn't have __NR_mmap2/__NR_fstat64 and uses
On Sun, Nov 11, 2012 at 1:37 PM, H.J. Lu hjl.to...@gmail.com wrote:
On Fri, Nov 9, 2012 at 9:23 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Fri, Nov 9, 2012 at 10:17 AM, H.J. Lu hjl.to...@gmail.com wrote:
Since x32 runs in 64-bit mode, for address -0x4300(%rax), hardware
sign-extends
On Tue, Nov 13, 2012 at 8:15 AM, H.J. Lu hjl.to...@gmail.com wrote:
Since x32 runs in 64-bit mode, for address -0x4300(%rax), hardware
sign-extends displacement from 32-bits to 64-bits and adds it to %rax.
But x32 wants 32-bit -0x4300, not 64-bit -0x4300. This patch
uses 32-bit
On Mon, Nov 12, 2012 at 10:13:08PM -0500, David Edelsohn wrote:
gcc/asan.c probably should have been split into two files because it
works at multiple levels. But given that it invokes
ASM_GENERATE_INTERNAL_LABEL, it needs to include tm_p.h to include
target-protos.h.
Committed as obvious
95 matches
Mail list logo