On 04/09/2012 11:17 PM, Adam Conrad wrote:
Like I said, then, you didn't actually read or understand why proposing
multilib paths doesn't work. You realize conceptually, I hope, that
there's no guarantee of uniqueness in lib/lib64/lib32/libsf/libhf once
you cross the base CPU architecture
So, I'd like to change all the ada testcases to use normal unix line endings.
testsuite/gnat.dg/taft_type2_pkg.ads
is an example if one such file, any objections?
No objections.
On Apr 9, 2012, at 11:24 PM, Arnaud Charlet wrote:
So, I'd like to change all the ada testcases to use normal unix line endings.
testsuite/gnat.dg/taft_type2_pkg.ads
is an example if one such file, any objections?
No objections.
Applied to trunk and 4.7.x. Thanks.
Index:
No patch review - but and answer to a question and a comment.
Janus Weil wrote:
I am aware that it will break the ABI, but only for programs involving
procedure pointers (which still is a 'relatively' new feature,
supported since gfortran 4.4).
Btw, speaking of ABI breaking: What are the
Hello Ayal,
First of all, thanks for your feedback. Now to your questions:
On 31.03.2012 3:20, Ayal Zaks wrote:
Roman, Andrey,
Sorry for the delayed response.
It would indeed be good to have SMS apply to more loop patterns, still
within the realm of *countable* loops. SMS was originally
Back on this! It turn out that this breaks the shared Ada runtime.
Indeed, exported variables on Ada packages in a DLLs are only accessible
when linking against DLL (thanks to runtime pseudo reloc).
With the patch applied it is not possible to build any Ada application
using the shared
The patch with correct configure output is ok.
Thanks - this is what I committed.
Is this something that can be considered for backporting to release
branches ? This patch technically doesn't fix a regression but brings
in line behaviour of the normal bootstrap for %gnu_unique_object ?
On Tue, 10 Apr 2012 07:36:07 +0200
Jakub Jelinek ja...@redhat.com wrote:
We really want consistency about the dynamic linker names etc. across
different targets and sneaking silently multiarch paths on one architecture
would make it inconsistent with all the others. So, please just use
On Apr 6, 2012, at 12:26 AM, Ian Lance Taylor wrote:
Tristan Gingold ging...@adacore.com writes:
gcc/
2012-04-05 Tristan Gingold ging...@adacore.com
* gengtype.c (main): Make uintptr_t a known type.
include/
2012-04-05 Tristan Gingold ging...@adacore.com
*
On Mon, 9 Apr 2012, Jan Hubicka wrote:
Hi,
this patch fixes several different ICEs related to handling aliases in WHOPR
partitioning. It took me over week debug this, but when variable alias
is added to a boundary and its destination is not added, we get queue of
unforutnate events where
On Mon, Apr 9, 2012 at 11:43 PM, Eric Botcazou ebotca...@adacore.com wrote:
Hi,
this adds generic support for __builtin_bswap16 (only PowerPC has it for now).
It is mapped to the bswap optab in HImode, whose implementation is as follows:
- if a bswaphi2 pattern is present (PowerPC), it is
I don't know enough about Fortran to know whether the same issues arise
there. Perhaps in Fortran a common symbol is always a common symbol and
can never be a defined symbol. If that is the case then for Fortran I
think it would be safe to change the alignment of the common symbol. Of
On Tue, Apr 10, 2012 at 2:45 AM, John David Anglin
d...@hiauly1.hia.nrc.ca wrote:
The current 4.5, 4.6 and 4.7 branches are now seriously broken for all
PA target due to the fix applied for PR middle-end/52640, an optimization
fix. This is because the PA backend defers output of function
On Tue, Apr 10, 2012 at 12:41 PM, Kai Tietz ktiet...@googlemail.com wrote:
Hello,
recent changes to cfgloop have caused an ICE on bootstrapping
libstdc++ for mingw targets. I assume same ICE happens for cygwin
hosted gcc bootstrap, too. ICE happens on compiling of
On Tue, Apr 10, 2012 at 1:00 PM, Dominique Dhumieres domi...@lps.ens.fr wrote:
I don't know enough about Fortran to know whether the same issues arise
there. Perhaps in Fortran a common symbol is always a common symbol and
can never be a defined symbol. If that is the case then for Fortran I
To cgraphunit.c, where its only caller (should) reside.
Bootstrapped and tested on x86_64-unknown-linux-gnu, applied.
Richard.
2012-04-10 Richard Guenther rguent...@suse.de
* toplev.h (tree_rest_of_compilation): Remove.
* tree-optimize.c (tree_rest_of_compilation): Likewise.
Hi,
Patch OK?
ChangeLog:
tree-parloops.c (eliminate_local_variables): Add braces to suppress warnings.
Index: tree-parloops.c
===
--- tree-parloops.c (revision 186243)
+++ tree-parloops.c (working copy)
@@ -723,13 +723,15
On 4/10/2012 1:35 AM, Mike Stump wrote:
So, I'd like to change all the ada testcases to use normal unix line endings.
testsuite/gnat.dg/taft_type2_pkg.ads
is an example if one such file, any objections?
As long as the test is not about line endings this seems fine.
On 10-Apr-12, at 7:06 AM, Richard Guenther wrote:
I can't immediately see how your description of the list of pending
externals
and the vector is deleted. pa.c keeps its own vector which
references the
decls and the only issue I see is that if you call assemble_external
after
processing
Hello world,
this patch effectively trims the spaces from the string on
list-directed reads. This avoids the large overhead on
processing these spaces when reading from long lines.
I didn't do this for internal units which are arrays because
this would need a separate calculation for each
On Tue, Apr 10, 2012 at 2:15 PM, JonY jo...@users.sourceforge.net wrote:
Hi,
Patch OK?
What kind of warning?
ChangeLog:
tree-parloops.c (eliminate_local_variables): Add braces to suppress warnings.
Index: tree-parloops.c
===
Ping.
On Apr 4, 2012 at 1:02, Ilya Enkovich enkovich@gmail.com wrote:
On Tue, Apr 3, 2012 at 3:49 AM, Ilya Enkovich enkovich@gmail.com wrote:
On 3/04/2012, at 2:16 AM, Ilya Enkovich wrote:
The point is that one can build a toolchain for i686-linux-gnu that will
support both 32-bit
On Tue, Apr 10, 2012 at 2:28 PM, Dave Anglin dave.ang...@bell.net wrote:
On 10-Apr-12, at 7:06 AM, Richard Guenther wrote:
I can't immediately see how your description of the list of pending
externals
and the vector is deleted. pa.c keeps its own vector which references
the
decls and the
Hi,
issue was that TARGET_EXCEPT_UNWIND_INFO wasn't set for i386
architectures. As mingw targets are using SjLj, it is necessary that
this hook is present.
ChangeLog
2012-04-10 Kai Tietz kti...@redhat.com
PR c++/52918
* common/config/i386/i386-common.c
On 28 March 2012 11:13, Richard Guenther richard.guent...@gmail.com wrote:
On Wed, Mar 28, 2012 at 11:57 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Mar 27, 2012 at 3:17 PM, Ramana Radhakrishnan
ramana.radhakrish...@linaro.org wrote:
And the patch is now attached
Ping.
On 30/03/12 12:15, Andrew Stubbs wrote:
On 28/02/12 16:20, Andrew Stubbs wrote:
Hi all,
This patch implements 64-bit immediate constant loads in NEON.
The current state is that you can load const_vector, but not const_int.
This is clearly not ideal. The result is a constant pool entry
Ulrich Weigand uweig...@de.ibm.com writes:
http://gcc.gnu.org/ml/gcc-patches/2012-03/msg01269.html
I don't like approving patches I had a hand in, but since Paolo
was happy, and since no-one else seems to care... OK. Thanks for
getting this working.
Richard
Oleg Endo oleg.e...@t-online.de wrote:
The attached patch adds HImode addressing support.
Tested against rev. 186243 with
sudo make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb,
Hi,
and this implements generally strided loads where the stride is a
loop-invariant (constant or ssa-name). We only do so if the load can't be
handled by interleaving groups. The implementation is fairly straight
forward:
for (i = 0; i n; i += stride)
... = array[i];
On 4/10/12 9:32 AM, Jan Hubicka wrote:
Hi,
LTO partitioning logic is one of places that are really symmetric accross most
types
of symbols; also the implementation is convoluted now by split in between lto.c
that
computes cgraph/varpool node sets specifying partitions and lto-cgraph.c that
On Tue, 10 Apr 2012, Jan Hubicka wrote:
Hi,
this is very basis of the symbol table work. The basic idea is:
1) give cgraph node and varpool node common base (symtab node)
2) Move common data there: that is type, declaration, visibility flags,
force_output, ipa references and other
On Tue, Apr 10, 2012 at 3:36 PM, Steven Bosscher stevenb@gmail.com wrote:
On Tue, Apr 10, 2012 at 2:44 PM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Apr 10, 2012 at 2:28 PM, Dave Anglin dave.ang...@bell.net wrote:
On 10-Apr-12, at 7:06 AM, Richard Guenther wrote:
I can't
On Tue, 2012-04-10 at 22:42 +0900, Kaz Kojima wrote:
Oleg Endo oleg.e...@t-online.de wrote:
The attached patch adds HImode addressing support.
Tested against rev. 186243 with
sudo make -k check RUNTESTFLAGS=--target_board=sh-sim
\{-m2/-ml,-m2/-mb,-m2a/-mb,-m2a-single/-mb,-m4/-ml,-m4/-mb,
Yeah. And you don't need to change the FEs in any way, all that is needed
is to change the middle-end/expansion (builtins.c - e.g. get_memmodel)
and the backend (plus predefine the macros in the backend).
Jakub
Hi Jakub,
Attached patch implements HLE support for
- Original Message -
BTW, do you have the numbers of CSiBE with this?
Only for -m4-single -ml -O2 -mpretend-cmove so far.
Not so spectacular :T
I'll also do a comparison of more variants to see if something went
really bad. It's a bit difficult to isolate the degradations because
The condition I orignally set myself was that this patch should only
go in if it becomes the default on at least one architecture,
specifically ARM. Ulrich tells me that Linaro have now made it
the default for ARM in their GCC 4.7 release, so hopefully Ramana
would be OK with doing the same
On Tue, 2012-04-10 at 07:14 -0700, Nathan Froyd wrote:
- Original Message -
BTW, do you have the numbers of CSiBE with this?
Only for -m4-single -ml -O2 -mpretend-cmove so far.
Not so spectacular :T
I'll also do a comparison of more variants to see if something went
really
On Tue, Apr 10, 2012 at 06:12:08PM +0400, Kirill Yukhin wrote:
Attached patch implements HLE support for __atomic_compare_exchange_n.
The target hook is definitely not appropriate, just define it in
ix86_target_macros in i386-c.c instead or so.
Jakub
On Tue, Apr 10, 2012 at 3:46 PM, Michael Matz m...@suse.de wrote:
Hi,
and this implements generally strided loads where the stride is a
loop-invariant (constant or ssa-name). We only do so if the load can't be
handled by interleaving groups. The implementation is fairly straight
forward:
On Tue, 10 Apr 2012 12:18:51 +0300
Konstantinos Margaritis konstantinos.margari...@linaro.org wrote:
On Tue, 10 Apr 2012 07:36:07 +0200
Jakub Jelinek ja...@redhat.com wrote:
We really want consistency about the dynamic linker names etc.
across different targets and sneaking silently
On Tue, Apr 10, 2012 at 3:06 PM, JonY jo...@users.sourceforge.net wrote:
On 4/10/2012 20:37, Richard Guenther wrote:
On Tue, Apr 10, 2012 at 2:15 PM, JonY wrote:
Hi,
Patch OK?
What kind of warning?
Oops, I forgot to mention gcc was complaining about missing braces.
Not for me. And I
On Tue, Apr 10, 2012 at 3:14 PM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Richard Guenther richard.guent...@gmail.com writes:
On Fri, Apr 6, 2012 at 10:13 AM, Richard Sandiford
rdsandif...@googlemail.com wrote:
Richard Guenther richard.guent...@gmail.com writes:
They can affect
On Thu, Apr 5, 2012 at 8:57 AM, H.J. Lu hjl.to...@gmail.com wrote:
On Thu, Apr 5, 2012 at 8:36 AM, Uros Bizjak ubiz...@gmail.com wrote:
On Thu, Apr 5, 2012 at 3:28 PM, H.J. Lu hjl.to...@gmail.com wrote:
Looking at how other targets implement this check, I don't think that
this is a problem at
On Tue, Apr 10, 2012 at 7:12 AM, Kirill Yukhin kirill.yuk...@gmail.com wrote:
Yeah. And you don't need to change the FEs in any way, all that is needed
is to change the middle-end/expansion (builtins.c - e.g. get_memmodel)
and the backend (plus predefine the macros in the backend).
On Tue, Apr 10, 2012 at 07:42:53AM -0700, H.J. Lu wrote:
Attached patch implements HLE support for __atomic_compare_exchange_n.
So, to emit HLE prefix, it is possible to do:
int
foo2 (int *p, int oldv, int newv)
{
__atomic_compare_exchange_n (p, oldv, newv, 0, __ATOMIC_ACQUIRE |
On Mon, Apr 9, 2012 at 6:11 PM, Eric Botcazou ebotca...@adacore.com wrote:
Isn't the gimple cfg-cleanup we run post optimization (right before
expansion) not enough? Or the cfg-cleanup we perform right after expansion
now? At least if the branches are really caused by the gimplification
On Thu, Apr 5, 2012 at 11:48 AM, H.J. Lu hongjiu...@intel.com wrote:
Hi,
CSE turns (reg:DI 64)
(insn 6 3 7 2 (set (reg:DI 64)
(sign_extend:DI (subreg/u:SI (reg/v/f:DI 63 [ addr ]) 0))) x.i:6 122
{*extendsidi2_rex64} (nil))
into (reg/f:DI 64 [ addr ]). But nonzero_bits1 in
This properly tells the CFG manipulation of the case where we thread
through a loop exit - the duplicated header block will end up in the
outer loop and the original loop structure will be preserved.
Bootstrapped on x86_64-unknown-linux-gnu, testing in progress.
Richard.
2012-04-10 Richard
Hello,
I am proposing a series of patches which is supposed to address the
remaining issues (I am aware of) preventing us from enabling the
-ftrack-macro-expansion by default.
The idea is to address each issue I notice in the course of trying to
bootstrap the compiler and running the tests with
Hi,
this patch speeds up polyhedrons protein on Bulldozer quite a bit. The
things is that in this testcase cshift is called with a very short length
(=3) and that the shift amount always is less than the length.
Surprisingly the division instruction takes up considerable amount of
time, so
Hello,
cpp_sys_macro_p crashes when -ftrack-macro-expansion is on. The issue
can be reproduced by running the tests:
runtest --tool gcc --tool_opts=-ftrack-macro-expansion cpp.exp=sysmac1.c
runtest --tool gcc --tool_opts=-ftrack-macro-expansion cpp.exp=sysmac2.c
This is because it just
RTL optimizers perform CFG adjustments themselves rather than relying
on cleanup_cfg (as the gimple level does). This makes it necessary
to handle things all-over-the-place. This case is if-conversion
performing forwarder block removal (well, it calls it speculation),
removing empty loop
This patch makes token pasting work with -ftrack-macro-expansion
turned on. It improves some pasting related tests of the gcc.dg/cpp
sub-directory.
Tested and bootstrapped on x86_64-unknown-linux-gnu against trunk.
Note that the bootstrap with -ftrack-macro-expansion exhibits other
separate
Disclaimer: I am sorry for the length of this message.
In essence, it's just a one liner fix in gengtype.c. I felt the need to
explain extensively what I think I understood because it didn't seem
obvious to me.
So, when -ftrack-macro-expansion is activated, the PCH generation
machinery can
On Tue, 10 Apr 2012 09:32:22 -0500
Dennis Gilmore den...@gilmore.net.au wrote:
every distro uses a unique triplet, by putting the triplet in there you
then need to get all distros to change to using the same triplets. I
personally prefer /libhfp rather than /libhf but I am ok with using
On Tue, Apr 10, 2012 at 4:53 PM, Michael Matz m...@suse.de wrote:
Hi,
this patch speeds up polyhedrons protein on Bulldozer quite a bit. The
things is that in this testcase cshift is called with a very short length
(=3) and that the shift amount always is less than the length.
Surprisingly
Sorry, for the noise. This patch just activates dw2 and therefore
avoids the ICE.
Continue on that.
Kai
It is a C-family FE function, not sure how it ended up in stmt.c.
OK to commit?
Cheers,
Manuel.
Bootstrapped and regression tested with languages=all,ada,obj-c++
2012-04-10 Manuel López-Ibáñez m...@gcc.gnu.org
* c-family/c-common.c (warn_if_unused_valuer): Move definition to here.
On Tue, Apr 10, 2012 at 09:32:22AM -0500, Dennis Gilmore wrote:
On Tue, 10 Apr 2012 12:18:51 +0300
Konstantinos Margaritis konstantinos.margari...@linaro.org wrote:
On Tue, 10 Apr 2012 07:36:07 +0200
Jakub Jelinek ja...@redhat.com wrote:
We really want consistency about the dynamic linker
Hi,
On Tue, 10 Apr 2012, Steven Bosscher wrote:
This is OK.
r186283.
Do you think it would be worthwhile to do this transformation in the
middle end too, based on profile information for values?
I'd think so, but it probably requires a new profiler that counts for how
often 0 = A = B for
On Tue, Apr 10, 2012 at 4:56 PM, Richard Guenther rguent...@suse.de wrote:
I'm sure more RTL optimiziation fallout will pop up - and I wonder
if we should simply avoid modifying the CFG all over the place and
instead do that in cleanup_cfg. Thus, in the if-conversion case,
simply do the
Hi,
On Tue, 10 Apr 2012, Richard Guenther wrote:
+ vec_inv = build_constructor (vectype, v);
+ new_temp = vect_init_vector (stmt, vec_inv, vectype, gsi);
+ new_stmt = SSA_NAME_DEF_STMT (new_temp);
+ mark_symbols_for_renaming (new_stmt);
This should not
On Tue, Apr 10, 2012 at 10:25 AM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
It is a C-family FE function, not sure how it ended up in stmt.c.
OK to commit?
OK.
H.J. Lu hjl.to...@gmail.com writes:
So, to emit HLE prefix, it is possible to do:
int
foo2 (int *p, int oldv, int newv)
{
__atomic_compare_exchange_n (p, oldv, newv, 0, __ATOMIC_ACQUIRE |
__ATOMIC_USE_HLE, __ATOMIC_ACQUIRE);
return oldv;
}
This is wrong since HLE ACQUIRE/RELEASE has
On 04/10/2012 09:35 AM, Richard Sandiford wrote:
Hi Vlad,
Back in Decemember, when we were still very much in stage 3, I sent
an RFC about an alternative implementation of -fsched-pressure.
Just wanted to send a reminder now that we're in the proper stage:
): New.
* common.opt (fdiagnostics-show-caret): New option.
* doc/invoke.texi (fdiagnostics-show-caret): Document it.
testsuite/
* lib/prune.exp: Add -fno-diagnostics-show-caret.
caret-diagnostics-20120410.diff
Description: Binary data
FWIW, my use case for multiarch is not sharing the root filesystem
among multiple systems. It's sharing the non-lib namespace (/etc,
/bin, data) among multiple instruction sets / ABI variants on the same
system. I don't need (/usr)?/s?bin to be decorated with a triplet,
because the kernel
On Tue, Apr 10, 2012 at 4:20 PM, Jakub Jelinek ja...@redhat.com wrote:
On Tue, Apr 10, 2012 at 06:12:08PM +0400, Kirill Yukhin wrote:
Attached patch implements HLE support for __atomic_compare_exchange_n.
The target hook is definitely not appropriate, just define it in
ix86_target_macros in
Hi,
issue was caused by the patch for PR middle-end/52772, which just
handles dw2. This patch makes sure that for sjlj we add new generated
bb's for dispatching table to loop, too.
ChangeLog
2012-04-10 Kai Tietz kti...@redhat.com
PR c++/52924
* except.c
On Mon, 2012-01-30 at 10:46 +0100, Richard Guenther wrote:
On Fri, Jan 27, 2012 at 5:56 PM, Peter Bergner berg...@vnet.ibm.com wrote:
This patch fixes PR16458 by using the type expression attached to a reg
rtx to detect its signedness and generating unsigned compares when
appropriate.
Hello!
This is wrong since HLE ACQUIRE/RELEASE has nothing to do with
C++ atomic acquire/release. You can have HLE RELEASE with C++
atomic acquire.
It makes sense to combine the two. On x86 C++ atomic acquire/release
means the compiler cannot move references outside. For HLE
we really
On 2012/04/05 18:53:28, asharif wrote:
Jan, please take a look and provide some feedback.
Thanks,
Ping?
http://codereview.appspot.com/5989046/
.. ah: remember to add PR libstdc++/52822 at the beginning of the
ChangeLog entry, otherwise Bugzilla will not pick your commit!
Thanks,
Paolo.
Hi All!
Here is a patch that enables unroll at O2 for Atom.
This gives good performance boost on EEMBC 2.0 (~+8% in Geomean for 32
bits) with quite moderate code size increase (~5% for EEMBC2.0, 32
bits).
Tested for i386 and x86-64, ok for trunk?
Thanks,
Igor
ChangeLog:
2012-04-10 Yakovlev
On Tue, Apr 10, 2012 at 1:36 PM, Peter Bergner berg...@vnet.ibm.com wrote:
2012-mm-dd Peter Bergner berg...@vnet.ibm.com
Michael Matz m...@suse.de
gcc/
PR target/16458
* rtlanal.c (unsigned_reg_p): New function.
* rtl.h (unsigned_reg_p): Prototype it.
On Tue, Apr 10, 2012 at 11:46 AM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
On 10 April 2012 00:28, Jason Merrill ja...@redhat.com wrote:
On 04/09/2012 04:01 PM, Manuel López-Ibáńez wrote:
* It uses the default cutoff as max_width, whatever it is (as
controlled by -fmessage-length).
Hi Richard, this is a follow up patch for more cleanups.
Bootstrap and tested on x86-64/linux.
Ok for trunk?
thanks,
David
todo_dump.p
Description: Binary data
cg
Description: Binary data
On Tue, Apr 10, 2012 at 10:38 AM, Richard Guenther
richard.guent...@gmail.com wrote:
On Tue, Apr 10, 2012 at 3:06 PM, JonY jo...@users.sourceforge.net wrote:
On 4/10/2012 20:37, Richard Guenther wrote:
On Tue, Apr 10, 2012 at 2:15 PM, JonY wrote:
Hi,
Patch OK?
What kind of warning?
On 04/10/2012 12:46 PM, Manuel López-Ibáñez wrote:
+ max_width = context-caret_max_width;
+ if (max_width= 0)
+max_width = INT_MAX;
I don't think we need the test here; diagnostic_set_caret_max_width
should make sure caret_max_width is set sensibly. Otherwise, yes, thanks.
On
On 10 April 2012 20:52, Gabriel Dos Reis g...@integrable-solutions.net wrote:
On Tue, Apr 10, 2012 at 11:46 AM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
On 10 April 2012 00:28, Jason Merrill ja...@redhat.com wrote:
On 04/09/2012 04:01 PM, Manuel López-Ibáńez wrote:
* It uses the
On 10 April 2012 21:31, Jason Merrill ja...@redhat.com wrote:
On 04/10/2012 12:46 PM, Manuel López-Ibáñez wrote:
+ max_width = context-caret_max_width;
+ if (max_width= 0)
+ max_width = INT_MAX;
I don't think we need the test here; diagnostic_set_caret_max_width should
make sure
Consider the test case gcc/testsuite/gcc.dg/debug/dwarf2/pr41445-5.c.
Its interesting part is:
#define A(x) vari x /* line 7. */
#define vari(x)
#define B , varj
int A(B) ; /* line 10. */
In its initial version, this test was being pre-processed as:
# 1
In this case, can we reverse this sentence and just emit lock
xacquire for MEMMODEL_ACQUIRE and lock xrelease for
MEMMODEL_RELEASE ? Do we need separate HLE_* defines or can we somehow
recycle existing C++11 memmodel defines?
No you absolutely can't. Transactions are quite different from a
Apparently, quite some places in the compiler (like the C/C++
preprocessor, the debug info machinery) expect expand_location to
resolve to locations that are in the main source file, even if the
token at stake comes from a macro that was defined in a header
somewhere. Turning on
Now that diagnostics for tokens coming from macro expansions point to
the spelling location of the relevant token (and then displays the
context of the expansion), some ugly (not so seldom) corner cases can
happen.
When the relevant token is a built-in token (which means the location
of that
On Tue, Apr 10, 2012 at 9:34 AM, Andi Kleen a...@firstfloor.org wrote:
H.J. Lu hjl.to...@gmail.com writes:
So, to emit HLE prefix, it is possible to do:
int
foo2 (int *p, int oldv, int newv)
{
__atomic_compare_exchange_n (p, oldv, newv, 0, __ATOMIC_ACQUIRE |
__ATOMIC_USE_HLE,
If it is the case, can we generate HLE RELEASE/ACQUIRE
prefix automatically for C++ atomics via -mhle command line option.
Then you don't need to modify the source codes to enable HLE support.
No, for HLE someone needs to decide whether HLE is beneficial for a given
lock. There are cases
2012/4/7 Jason Merrill ja...@redhat.com:
On 04/07/2012 11:37 AM, Fabien Chêne wrote:
Perhaps it is more correct like that, in cp_parser_set_decl_spec_type ?
Even that seems late. Why not just return the target decl from
cp_parser_class_name?
Ah yes, that's slightly better.
(I've kept the
On Tue, Apr 3, 2012 at 6:56 PM, Joseph S. Myers jos...@codesourcery.com wrote:
(e) Existing practice for cases that do use different dynamic linkers is
to use a separate library directory, not just dynamic linker name, as in
lib32 and lib64 for MIPS or libx32 for x32; it's certainly a lot
Besides the warning emitted by warn_uninit, this function wants
to hint the user at where the uninitialized variable was declared, for
cases where the declaration location is outside the current function.
Now that expand_location expands to the location that is in the main
source file (even for
On 10 April 2012 21:41, Manuel López-Ibáñez lopeziba...@gmail.com wrote:
On 10 April 2012 21:31, Jason Merrill ja...@redhat.com wrote:
On 04/10/2012 12:46 PM, Manuel López-Ibáñez wrote:
+ max_width = context-caret_max_width;
+ if (max_width= 0)
+ max_width = INT_MAX;
I don't think we
On Apr 10, 2012, at 5:24 AM, Robert Dewar wrote:
On 4/10/2012 1:35 AM, Mike Stump wrote:
So, I'd like to change all the ada testcases to use normal unix line endings.
testsuite/gnat.dg/taft_type2_pkg.ads
is an example if one such file, any objections?
As long as the test is not about
On Apr 10, 2012, at 11:52 AM, Gabriel Dos Reis wrote:
On Tue, Apr 10, 2012 at 11:46 AM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
On 10 April 2012 00:28, Jason Merrill ja...@redhat.com wrote:
On 04/09/2012 04:01 PM, Manuel López-Ibáńez wrote:
* It uses the default cutoff as
On Tue, Apr 10, 2012 at 5:23 PM, Mike Stump mikest...@comcast.net wrote:
On Apr 10, 2012, at 11:52 AM, Gabriel Dos Reis wrote:
On Tue, Apr 10, 2012 at 11:46 AM, Manuel López-Ibáñez
lopeziba...@gmail.com wrote:
On 10 April 2012 00:28, Jason Merrill ja...@redhat.com wrote:
On 04/09/2012 04:01
On Apr 10, 2012, at 1:45, Mike Stump mikest...@comcast.net wrote:
I assume that was a friendly, please feel free to fix *.ads as well.
Yes, sorry for the terse email. I wasn't quite sure if your message implied
there was only an issue with *.adb or not and wasn't in a position to check at
On Apr 10, 2012, at 4:37 PM, Gabriel Dos Reis wrote:
You should check the environment first
I don't think we should have a knob for this.
This is fully mystifying to me and I still do not understand what it is all
about after reading it 8 times.
Let me rephrase, I disagree, we should not
On 04/10/2012 04:33 PM, Manuel López-Ibáñez wrote:
A new version of the patch. It removes the special handling of tabs.
And abstracts out the adjusting of the line as Gabriel suggested. I am
not sure what else could be abstracted out. The function is much
shorter than other functions in
On 04/10/2012 04:23 PM, Fabien Chêne wrote:
Ah yes, that's slightly better.
(I've kept the NULL check in strip_using_decl, it seems safer to me.
Just tell me if you prefer not)
Tested x86_64-unkown-linux-gnu. OK for trunk and 4.7 ?
2012-03-08 Fabien Chênefab...@gcc.gnu.org
PR
On Tuesday 10 April 2012 01:17:36 Adam Conrad wrote:
On Tue, Apr 10, 2012 at 12:01:57AM -0400, Mike Frysinger wrote:
On Monday 09 April 2012 19:31:40 Adam Conrad wrote:
I realize that most people can't see past their own use case to
understand why a unique location for linkers is helpful,
On Tuesday 10 April 2012 12:46:49 Michael Edwards wrote:
That way I can grandfather in binaries with ABI-ignorant
hard-coded library paths, and still handle ISA variants. The
extranoise might be neon, or ssse3
aren't ISA variants handled already by glibc ? that's what the hwcaps stuff
does
1 - 100 of 105 matches
Mail list logo