On Mon, Aug 22, 2011 at 5:06 AM, Dimitrios Apostolou wrote:
> Hello again,
>
> most of this patch was posted at the beginning of my GSOC adventure and
> primarily is about replacing fprintf() calls with custom faster ones. From
> that time I changed minor things you suggested, omitted some complex
Hello!
2011-08-23 Uros Bizjak
* config/i386/64/sfp-machine.h (ASM_INVALID): New define.
(ASM_DIVZERO): Ditto.
(FP_HANLDE_EXCEPTIONS): Use ASM_INVALID and ASM_DIVZERO.
Tested on x86_64-pc-linux-gnu {,-m32}. Committed to mainline SVN, will
be backported to release branch
On 08/22/11 22:39:04, Joseph S. Myers wrote:
[...]
> This isn't a type modifier; neither is __builtin_types_compatible_p. It's
> not within the first 28.
[...]
> I don't believe the comment is accurate; I'm not aware of any code for any
> C-family front end that uses these values as mask bits at
On Mon, 22 Aug 2011, Gary Funck wrote:
>
> On 08/19/11 15:55:12, Joseph S. Myers wrote:
> > Index: gcc/c-family/c-common.h
> > ===
> > --- gcc/c-family/c-common.h (revision 177894)
> > +++ gcc/c-family/c-common.h (working copy)
> > @
On 08/19/11 15:55:12, Joseph S. Myers wrote:
> Index: gcc/c-family/c-common.h
> ===
> --- gcc/c-family/c-common.h (revision 177894)
> +++ gcc/c-family/c-common.h (working copy)
> @@ -103,7 +103,7 @@ enum rid
>/* C extensions *
On 08/22/2011 02:54 AM, Torvald Riegel wrote:
> Move x86 TLS definition to linux/x86.
>
> * config/x86/tls.h: Moved to ...
> * config/linux/x86/tls.h: ... here.
Ok.
r~
On 08/22/2011 02:42 AM, Torvald Riegel wrote:
> Use __x86_64__ instead of __LP64__ in setjmp/longjmp and TLS
> definitions.
>
> H.J.: Is that sufficient for x32, or do we need entirely different code?
> If so, can you please provide the required changes?
The SJLJ part should be ok for x32.
The T
Dear all,
this patch added token/offset support for assumed-shape coarray dummies
(with .-fcoarray=lib).
Build and regtested.
OK for the trunk?
* * *
BACKGROUND
For coarrays with -fcoarray=lib, for remote access, one needs to know
two things: token and offset.
a) A token (of type "void
On Mon, Aug 22, 2011 at 9:38 PM, Kirill Yukhin wrote:
> Thanks for inputs.
> Updated test attached.
>>> testsuite/ChangeLog entry:
>>> 2011-08-22 Kirill Yukhin
>>>
>>> PR target/50155
>>> * gcc.target/i386/pr50155.c: New test.
>>
>> For the testcase, I think you want explicit -
On Mon, Aug 22, 2011 at 10:49 PM, Artem Shinkarov
wrote:
> On Mon, Aug 22, 2011 at 9:42 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 5:58 PM, Artem Shinkarov
>> wrote:
>>> On Mon, Aug 22, 2011 at 4:50 PM, Richard Guenther
>>> wrote:
On Mon, Aug 22, 2011 at 5:43 PM, Artem Shinka
On Mon, Aug 22, 2011 at 10:48 PM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 1:46 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 10:39 PM, H.J. Lu wrote:
>>> On Mon, Aug 22, 2011 at 1:34 PM, Richard Guenther
>>> wrote:
On Mon, Aug 22, 2011 at 6:02 PM, H.J. Lu wrote:
> On Mon,
On Mon, Aug 22, 2011 at 9:42 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 5:58 PM, Artem Shinkarov
> wrote:
>> On Mon, Aug 22, 2011 at 4:50 PM, Richard Guenther
>> wrote:
>>> On Mon, Aug 22, 2011 at 5:43 PM, Artem Shinkarov
>>> wrote:
On Mon, Aug 22, 2011 at 4:34 PM, Richard Guent
On Mon, Aug 22, 2011 at 1:46 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 10:39 PM, H.J. Lu wrote:
>> On Mon, Aug 22, 2011 at 1:34 PM, Richard Guenther
>> wrote:
>>> On Mon, Aug 22, 2011 at 6:02 PM, H.J. Lu wrote:
On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
> Hi,
>>>
On Mon, Aug 22, 2011 at 10:39 PM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 1:34 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 6:02 PM, H.J. Lu wrote:
>>> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
Hi,
On Mon, 22 Aug 2011, H.J. Lu wrote:
> >> Oh, I th
On Mon, Aug 22, 2011 at 8:50 PM, Uros Bizjak wrote:
> On Mon, Aug 22, 2011 at 5:34 PM, Richard Guenther
> wrote:
>
>>> In this case it is simple to analyse that a is a comparison, but you
>>> cannot embed the operations of a into VEC_COND_EXPR.
>>
>> Sure, but if the above is C source the fronten
On Mon, Aug 22, 2011 at 5:58 PM, Artem Shinkarov
wrote:
> On Mon, Aug 22, 2011 at 4:50 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 5:43 PM, Artem Shinkarov
>> wrote:
>>> On Mon, Aug 22, 2011 at 4:34 PM, Richard Guenther
>>> wrote:
On Mon, Aug 22, 2011 at 5:21 PM, Artem Shinkar
On Mon, Aug 22, 2011 at 9:50 PM, Uros Bizjak wrote:
> On Mon, Aug 22, 2011 at 5:34 PM, Richard Guenther
> wrote:
>
>>> In this case it is simple to analyse that a is a comparison, but you
>>> cannot embed the operations of a into VEC_COND_EXPR.
>>
>> Sure, but if the above is C source the fronten
On Mon, Aug 22, 2011 at 1:34 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 6:02 PM, H.J. Lu wrote:
>> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
>>> Hi,
>>>
>>> On Mon, 22 Aug 2011, H.J. Lu wrote:
>>>
>> Oh, I thought it was data initialized by the constructor ...
>
>
On Mon, Aug 22, 2011 at 11:59 AM, Joseph S. Myers
wrote:
> On Mon, 22 Aug 2011, H.J. Lu wrote:
>
>> > Require a good assembler on ELF targets and just enable this by default
>> > for them without trying a configure test that won't work for cross
>> > compilation (AC_RUN_IFELSE is bad).
>> >
>> > T
On Mon, Aug 22, 2011 at 6:02 PM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
>> Hi,
>>
>> On Mon, 22 Aug 2011, H.J. Lu wrote:
>>
>>> >> Oh, I thought it was data initialized by the constructor ...
>>> >
>>> > Sriramans patch right now has a function __cpu_indicator_init
On Mon, Aug 22, 2011 at 5:34 PM, Richard Guenther
wrote:
>> In this case it is simple to analyse that a is a comparison, but you
>> cannot embed the operations of a into VEC_COND_EXPR.
>
> Sure, but if the above is C source the frontend would generate
> res = a != 0 ? v0 : v1; initially. An opti
OK.
Jason
Thanks for inputs.
Updated test attached.
K
On Mon, Aug 22, 2011 at 11:12 PM, Jakub Jelinek wrote:
> On Mon, Aug 22, 2011 at 10:51:19PM +0400, Kirill Yukhin wrote:
>> testsuite/ChangeLog entry:
>> 2011-08-22 Kirill Yukhin
>>
>> PR target/50155
>> * gcc.target/i386/pr50155.c: N
On Mon, Aug 22, 2011 at 6:25 PM, Ilya Tocar wrote:
>> You don't need to add "negated" versions, one FMA builtin per mode is
>> enough, please see existing FMA4 descriptions. Just put unary minus
>> sign in the intrinsics header for "negated" operand and let GCC do its
>> job. Please see existing
> OK to install?
>
> * config/m32c/m32c.h (CLASS_MAX_NREGS): Remove macro.
> * config/m32c/m32c-protos.h (m32c_class_max_nregs): Remove.
> * config/m32c/m32c.c (m32c_class_max_nregs): Make static. Change
> regclass argument type to reg_class_t. Change 'max' and '
On Mon, Aug 22, 2011 at 10:51:19PM +0400, Kirill Yukhin wrote:
> testsuite/ChangeLog entry:
> 2011-08-22 Kirill Yukhin
>
> PR target/50155
> * gcc.target/i386/pr50155.c: New test.
For the testcase, I think you want explicit -mno-avx2 if
you want to check that vpaddd hasn't been
Hi.
This patch removes obsolete CLASS_MAX_NREGS macro from M32C back end in the
GCC and introduces equivalent TARGET_CLASS_MAX_NREGS target hooks.
Regression tested on m32c-unknown-elf.
OK to install?
* config/m32c/m32c.h (CLASS_MAX_NREGS): Remove macro.
* config/m32c/m32
On Mon, Aug 22, 2011 at 8:51 PM, Kirill Yukhin wrote:
> Attached fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50155
>
> ChangeLog entry:
> 2011-08-22 Kirill Yukhin
>
> PR target/50155
> * config/i386/sse.md (VI1248_AVX2): New.
> (3): Update.
> (*3): Likewise.
On Mon, Aug 22, 2011 at 11:58 AM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 11:50 AM, Sriraman Tallam wrote:
>> On Mon, Aug 22, 2011 at 9:02 AM, H.J. Lu wrote:
>>> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
Hi,
On Mon, 22 Aug 2011, H.J. Lu wrote:
> >> Oh, I thou
On Mon, 22 Aug 2011, H.J. Lu wrote:
> > Require a good assembler on ELF targets and just enable this by default
> > for them without trying a configure test that won't work for cross
> > compilation (AC_RUN_IFELSE is bad).
> >
> > The toplevel config/elf.m4 provides a good notion of what is or is
On Mon, Aug 22, 2011 at 11:50 AM, Sriraman Tallam wrote:
> On Mon, Aug 22, 2011 at 9:02 AM, H.J. Lu wrote:
>> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
>>> Hi,
>>>
>>> On Mon, 22 Aug 2011, H.J. Lu wrote:
>>>
>> Oh, I thought it was data initialized by the constructor ...
>
>
On Mon, Aug 22, 2011 at 11:53 AM, Joseph S. Myers
wrote:
> On Mon, 22 Aug 2011, H.J. Lu wrote:
>
>> On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
>> nd/or add another test to it that tests
>> > that you can actually use
>> > .section .init_array
>> > and it will use correct section flags
On 11-08-21 18:14 , Mark Heffernan wrote:
Ping?
Mark
On Fri, Aug 12, 2011 at 9:41 AM, Mark Heffernan wrote:
This patch adds an option for enabling/disabling the warning for
attempting to free nonheap objects (PR/38509). The warning is
imprecise and can issue false positives.
Bootstrapped on
On Mon, 22 Aug 2011, H.J. Lu wrote:
> On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
> nd/or add another test to it that tests
> > that you can actually use
> > .section .init_array
> > and it will use correct section flags for the section.
> >
>
> We need this information in config.gcc.
Hi,
Attached fix for http://gcc.gnu.org/bugzilla/show_bug.cgi?id=50155
ChangeLog entry:
2011-08-22 Kirill Yukhin
PR target/50155
* config/i386/sse.md (VI1248_AVX2): New.
(3): Update.
(*3): Likewise.
(_andnot3): Likewise.
(avx2_pbroadcast): Likewi
On Mon, Aug 22, 2011 at 9:02 AM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
>> Hi,
>>
>> On Mon, 22 Aug 2011, H.J. Lu wrote:
>>
>>> >> Oh, I thought it was data initialized by the constructor ...
>>> >
>>> > Sriramans patch right now has a function __cpu_indicator_init
On 11-08-22 14:20 , Gabriel Charette wrote:
2011-08-22 Gabriel Charette
* pph-streamer-in.c (pph_loc_offset): Add FIXME to move this variable
to pph_stream.encoder.r
(pph_in_include): New.
(pph_in_line_table_and_includes): Remove LINETAB parameter. Update
This is a refactoring patch, it doesn't change/fix anything in pph itself.
I extracted out/in logic for includes into their own functions.
I removed the LINETAB parameter from in/out functions for the line_table as
there is only one line_table and that's the only one we stream, the main/global
DJ Delorie writes:
>> Do I need to sync the config and libiberty parts to src manually or does
>> this happen by some sort of magic?
>
>
> intl/; config.rhost; libiberty/; libiberty's part of include/
> gcc: http://gcc.gnu.org
> Changes need to be done in tandem with the official GCC
On Mon, Aug 22, 2011 at 07:49:43PM +0200, Steven Bosscher wrote:
> On Mon, Aug 22, 2011 at 1:54 PM, Jakub Jelinek wrote:
> >> * From a very wide field of view, all this DF solving reminded me a
> >> lot of what I've seen in df-*.c. Why can't variable tracking be
> >> integrated in the main DF pass
On Mon, Aug 22, 2011 at 1:54 PM, Jakub Jelinek wrote:
>> * From a very wide field of view, all this DF solving reminded me a
>> lot of what I've seen in df-*.c. Why can't variable tracking be
>> integrated in the main DF pass of the compiler, the one that happens
>> *after* register allocation?
>
On 08/21/2011 02:14 AM, Matthias Klose wrote:
On 08/20/2011 09:51 PM, Matthias Klose wrote:
Multiarch [1] is the term being used to refer to the capability of a system to
install and run applications of multiple different binary targets on the same
system. The idea and name of multiarch date
Ping 2 ?
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg01889.html
http://gcc.gnu.org/ml/gcc-patches/2011-07/msg02555.html
On Mon, Aug 22, 2011 at 10:09 AM, H.J. Lu wrote:
> On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
> nd/or add another test to it that tests
>> that you can actually use
>> .section .init_array
>> and it will use correct section flags for the section.
>>
>
> We need this information in con
-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1
On 08/22/11 11:16, Dimitrios Apostolou wrote:
>
> Any updates will come as a followup to this thread. I still have to
> do some testing on other platforms. Do we have access to any PA and
> MIPS machinery? I also wanted to test on architecture with l
> Do I need to sync the config and libiberty parts to src manually or does
> this happen by some sort of magic?
intl/; config.rhost; libiberty/; libiberty's part of include/
gcc: http://gcc.gnu.org
Changes need to be done in tandem with the official GCC
sources or submit
Paolo Bonzini writes:
> On 08/19/2011 09:11 PM, Rainer Orth wrote:
>>
>> 2011-07-31 Rainer Orth
>>
>> config:
>> * picflag.m4: New file.
>>
>> gcc:
>> * configure.ac (GCC_PICFLAG_FOR_TARGET): Call it.
>> (PICFLAG_FOR_TARGET): Substitute.
>> * aclocal.m4: Regenerate.
On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
nd/or add another test to it that tests
> that you can actually use
> .section .init_array
> and it will use correct section flags for the section.
>
We need this information in config.gcc. But config.gcc is used
before assembler and readelf
Diego Novillo writes:
> On 11-08-22 07:10 , Dodji Seketeli wrote:
[...]
>> gcc/
>>
>> * c-family/c-pch.c (c_common_read_pch): Re-set line table right
>> after reading in the pch.
>
> OK.
Thanks, committed to revision r177964.
--
Dodji
On Mon, Aug 22, 2011 at 8:22 AM, Diego Novillo wrote:
>
> This patch fixes some template test cases. We were trying to
> expand functions that did not really need expanding (templates). This
> got me thinking that we are not approaching the expansion properly.
>
> We are trying to re-create all
On Mon, Aug 22, 2011 at 8:56 AM, Michael Matz wrote:
> Hi,
>
> On Mon, 22 Aug 2011, H.J. Lu wrote:
>
>> >> Oh, I thought it was data initialized by the constructor ...
>> >
>> > Sriramans patch right now has a function __cpu_indicator_init which is
>> > called from (adhoc constructed) ctors and th
On Mon, Aug 22, 2011 at 4:50 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 5:43 PM, Artem Shinkarov
> wrote:
>> On Mon, Aug 22, 2011 at 4:34 PM, Richard Guenther
>> wrote:
>>> On Mon, Aug 22, 2011 at 5:21 PM, Artem Shinkarov
>>> wrote:
On Mon, Aug 22, 2011 at 4:01 PM, Richard Guent
Hi,
On Mon, 22 Aug 2011, H.J. Lu wrote:
> >> Oh, I thought it was data initialized by the constructor ...
> >
> > Sriramans patch right now has a function __cpu_indicator_init which is
> > called from (adhoc constructed) ctors and that initializes variables
> > __cpu_model and __cpu_features ;-)
@@ -4354,7 +4354,8 @@ get_bit_range (unsigned HOST_WIDE_INT *b
|| TREE_CODE (innerdecl) == TARGET_MEM_REF)
&& !ptr_deref_may_alias_global_p (TREE_OPERAND (innerdecl, 0)))
|| (DECL_P (innerdecl)
- && (DECL_THREAD_LOCAL_P (innerdecl)
+ && ((TREE_CODE (inne
On Sun, Aug 21, 2011 at 4:19 PM, David Edelsohn wrote:
> This patch broke bootstrap on AIX. It emits a ".section" op in
> assembly but ".section" is an ELF syntax op not AIX XCOFF.
>
> FE..initialize_critical:
> .section .init_array
>
> varasm.c should not be generating ELF ops for
On Mon, Aug 22, 2011 at 5:43 PM, Artem Shinkarov
wrote:
> On Mon, Aug 22, 2011 at 4:34 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 5:21 PM, Artem Shinkarov
>> wrote:
>>> On Mon, Aug 22, 2011 at 4:01 PM, Richard Guenther
>>> wrote:
On Mon, Aug 22, 2011 at 2:05 PM, Artem Shinkar
Hi,
On Mon, 22 Aug 2011, Richard Guenther wrote:
> > Some functions are extremely large though. Do you mean that MEM
> > itself would be enlarged to have the MEM_ATTRS field so that one
> > operand is the address, then expr, then HWI size, offset, etc.?
> > Because if the mem attrs aren't sh
On Mon, Aug 22, 2011 at 4:34 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 5:21 PM, Artem Shinkarov
> wrote:
>> On Mon, Aug 22, 2011 at 4:01 PM, Richard Guenther
>> wrote:
>>> On Mon, Aug 22, 2011 at 2:05 PM, Artem Shinkarov
>>> wrote:
On Mon, Aug 22, 2011 at 12:25 PM, Richard Guen
On Mon, 22 Aug 2011, Dimitrios Apostolou wrote:
Hi Steven,
On Mon, 1 Aug 2011, Steven Bosscher wrote:
On Sun, Jul 31, 2011 at 11:59 PM, Steven Bosscher
wrote:
On Fri, Jul 29, 2011 at 11:48 PM, Steven Bosscher
wrote:
I'll see if I can test the patch on the compile farm this weekend,
just to
On Mon, Aug 22, 2011 at 5:21 PM, Artem Shinkarov
wrote:
> On Mon, Aug 22, 2011 at 4:01 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 2:05 PM, Artem Shinkarov
>> wrote:
>>> On Mon, Aug 22, 2011 at 12:25 PM, Richard Guenther
>>> wrote:
On Mon, Aug 22, 2011 at 12:53 AM, Artem Shink
This patch refactors pph_write_tree and pph_read_tree so that we can
reduce the amount of special casing we were doing. It first tries to
handle nodes based on their tree code class. The only class that
cannot be handled this way is tcc_exceptional, so we only deal with
those nodes individually.
On Mon, Aug 22, 2011 at 4:01 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 2:05 PM, Artem Shinkarov
> wrote:
>> On Mon, Aug 22, 2011 at 12:25 PM, Richard Guenther
>> wrote:
>>> On Mon, Aug 22, 2011 at 12:53 AM, Artem Shinkarov
>>> wrote:
Richard
I formalized an approach a
This patch fixes some template test cases. We were trying to
expand functions that did not really need expanding (templates). This
got me thinking that we are not approaching the expansion properly.
We are trying to re-create all the cgraph creation done during the
compilation of the header fi
On 08/22/2011 04:45 PM, H.J. Lu wrote:
Can I check in this patch to address AIX issue first?
I will submit a patch to test ".section .init_array" later?
Thanks.
Yes.
Paolo
On Mon, Aug 22, 2011 at 2:05 PM, Artem Shinkarov
wrote:
> On Mon, Aug 22, 2011 at 12:25 PM, Richard Guenther
> wrote:
>> On Mon, Aug 22, 2011 at 12:53 AM, Artem Shinkarov
>> wrote:
>>> Richard
>>>
>>> I formalized an approach a little-bit, now it works without target
>>> hooks, but some polishin
On Mon, Aug 22, 2011 at 4:22 PM, Jakub Jelinek wrote:
> Hi!
>
> The following testcase ICEs, because gsi_end_p (*gsi) and thus
> there is no stmt after it from which to copy over the location.
> As can be seen in the PR, we could do ugly hacks to retrieve locus
> from previous stmt (non-debug of c
On Mon, Aug 22, 2011 at 4:18 PM, Jakub Jelinek wrote:
> Hi!
>
> DECL_THREAD_LOCAL_P may be used only on VAR_DECLs, not other decls
> like PARM_DECL, RESULT_DECL etc.
> Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
> ok for trunk?
Ok.
Thanks,
Richard.
> 2011-08-22 Jakub J
On Mon, Aug 22, 2011 at 7:06 AM, H.J. Lu wrote:
> On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
>> On Sun, Aug 21, 2011 at 05:09:59PM -0700, H.J. Lu wrote:
>>> I didn't know .init_array section was enabled for AIX. Does this patch
>>> work for you?
>>
>> Some ELF targets (e.g. arm*-linu
On Mon, Aug 22, 2011 at 08:16:45AM -0600, Tom Tromey wrote:
> > "Jakub" == Jakub Jelinek writes:
>
> Jakub> For ccache and friends I think it would be better to have a
> Jakub> preprocessing mode that would output all lines as is (i.e. no
> Jakub> macro replacement), except for processing #in
On 11-08-22 07:10 , Dodji Seketeli wrote:
Hello,
In c_common_read_pch when gt_pch_restore loads a new pch, the previous
line table (referenced from the global 'line_table') is
garbage-collected and a new one is built.
As the global instance of cpp_reader referenced by the local variable
'pfile'
Hi!
TYPE_TRANSPARENT_AGGR types are passed and mangled as its first field.
But if somebody errorneously doesn't put any types in its definition,
the FE marks it TYPE_TRANSPARENT_AGGR early (e.g. even on a forward
declaration), but during mangling or in middle-end when trying to find out
how it wil
Hi!
The following testcase ICEs, because gsi_end_p (*gsi) and thus
there is no stmt after it from which to copy over the location.
As can be seen in the PR, we could do ugly hacks to retrieve locus
from previous stmt (non-debug of course) instead, but I'm probably missing
something obvious why we
Hi!
DECL_THREAD_LOCAL_P may be used only on VAR_DECLs, not other decls
like PARM_DECL, RESULT_DECL etc.
Fixed thusly, bootstrapped/regtested on x86_64-linux and i686-linux,
ok for trunk?
2011-08-22 Jakub Jelinek
PR middle-end/50141
* expr.c (get_bit_range): Only use DECL_THREA
Hi!
As the testcase below shows (on i686-linux or x86_64-linux -m32),
we don't unshare expressions in CALL_INSN_FUNCTION_USAGE, which
with entry_value support now include MEMs. The invalid sharing then
can lead to changes in unrelated insns affecting a call insn (or vice
versa), which results in
> "Jakub" == Jakub Jelinek writes:
Jakub> For ccache and friends I think it would be better to have a
Jakub> preprocessing mode that would output all lines as is (i.e. no
Jakub> macro replacement), except for processing #include/#include_next
Jakub> directives.
That exists -- -fdirectives-on
On Mon, Aug 22, 2011 at 2:57 PM, Mateusz Grabowski wrote:
>
>
> Richard Guenther-2 wrote:
>>
>>
>> The latter.
>>
>>
>
> But how to do it? I want to have all functions after SSA pass, but before
> any optimizations. Maybe you could tell me how to enforce it (or even better
> - a small example)?
>
On Mon, Aug 22, 2011 at 7:07 AM, Michael Matz wrote:
> Hi,
>
> On Sun, 21 Aug 2011, Richard Guenther wrote:
>
>> On Sat, Aug 20, 2011 at 11:02 PM, Richard Henderson wrote:
>> > On 08/19/2011 02:04 AM, Richard Guenther wrote:
>> >> So make sure that __cpu_indicator initially has a conservative cor
Hi,
On Sun, 21 Aug 2011, Richard Guenther wrote:
> On Sat, Aug 20, 2011 at 11:02 PM, Richard Henderson wrote:
> > On 08/19/2011 02:04 AM, Richard Guenther wrote:
> >> So make sure that __cpu_indicator initially has a conservative correct
> >> value? I'd still prefer the constructor-in-libgcc op
On Sun, Aug 21, 2011 at 10:37 PM, Jakub Jelinek wrote:
> On Sun, Aug 21, 2011 at 05:09:59PM -0700, H.J. Lu wrote:
>> I didn't know .init_array section was enabled for AIX. Does this patch
>> work for you?
>
> Some ELF targets (e.g. arm*-linux*) don't use elfos.h. IMHO you should
> instead add
>
http://gcc.gnu.org/viewcvs?view=revision&revision=177954
Committed that patchlet as obvious.
testsuite/
* gcc.dg/pr49994-2.c: Add dg-require-effective-target scheduling.
* gcc.dg/pr49994-3.c: Ditto.
Johann
Index: ChangeLog
===
Thanks!
K
On Mon, Aug 22, 2011 at 5:57 PM, H.J. Lu wrote:
> On Mon, Aug 22, 2011 at 6:18 AM, Kirill Yukhin
> wrote:
>> Hi,
>> thanks for input, Uros. Spaces were fixed.
>>
>> Updated patch is attached. ChangeLog entry is attached.
>>
>> Could anybody please commit it?
>>
>
> I checked in for y
On Mon, Aug 22, 2011 at 6:18 AM, Kirill Yukhin wrote:
> Hi,
> thanks for input, Uros. Spaces were fixed.
>
> Updated patch is attached. ChangeLog entry is attached.
>
> Could anybody please commit it?
>
I checked in for you.
--
H.J.
Hi Steven,
On Mon, 1 Aug 2011, Steven Bosscher wrote:
On Sun, Jul 31, 2011 at 11:59 PM, Steven Bosscher wrote:
On Fri, Jul 29, 2011 at 11:48 PM, Steven Bosscher wrote:
I'll see if I can test the patch on the compile farm this weekend,
just to be sure.
Bootstrap on
ia64-unknown-linux-gnu is
On Mon, Aug 22, 2011 at 2:42 AM, Torvald Riegel wrote:
> Use __x86_64__ instead of __LP64__ in setjmp/longjmp and TLS
> definitions.
>
> H.J.: Is that sufficient for x32, or do we need entirely different code?
> If so, can you please provide the required changes?
>
I need to take a look.
Thanks.
Richard Guenther-2 wrote:
>
>
> The latter.
>
>
But how to do it? I want to have all functions after SSA pass, but before
any optimizations. Maybe you could tell me how to enforce it (or even better
- a small example)?
Thanks in advance.
--
View this message in context:
http://old.nabble.
On Mon, Aug 22, 2011 at 2:21 PM, Mateusz Grabowski wrote:
>
> Thank you for all the answers. Now I have another problem.
>
> When I reach a function call during statement iteration, I want to go to
> this function's code.
>
> if (is_gimple_call(stmt)) {
> tree fndecl = gimple_call_f
Thank you for all the answers. Now I have another problem.
When I reach a function call during statement iteration, I want to go to
this function's code.
if (is_gimple_call(stmt)) {
tree fndecl = gimple_call_fndecl(stmt); // This returns
function_decl ;-)
Hello again,
most of this patch was posted at the beginning of my GSOC adventure and
primarily is about replacing fprintf() calls with custom faster ones. From
that time I changed minor things you suggested, omitted some complexities
that offered minor speed-up, and made the code as clear as p
On Mon, Aug 22, 2011 at 12:25 PM, Richard Guenther
wrote:
> On Mon, Aug 22, 2011 at 12:53 AM, Artem Shinkarov
> wrote:
>> Richard
>>
>> I formalized an approach a little-bit, now it works without target
>> hooks, but some polishing is still required. I want you to comment on
>> the several import
On Mon, Aug 22, 2011 at 01:30:33PM +0300, Dimitrios Apostolou wrote:
> --- gcc/var-tracking.c2011-06-03 01:42:31 +
> +++ gcc/var-tracking.c2011-08-22 09:52:08 +
> @@ -1069,14 +1067,14 @@ adjust_insn (basic_block bb, rtx insn)
> static inline bool
> dv_is_decl_p (decl_or_va
On Mon, Aug 22, 2011 at 1:37 PM, Dimitrios Apostolou wrote:
> I should note here that specialised hash-tables in pointer-set.c have a
> load-factor of at most 25%. Also another very fast hash table I've studied,
> dense_hash_map from google's sparse_hash_table, has a load factor of 50%
> max.
>
>
On Mon, Aug 22, 2011 at 02:26:58PM +0300, Dimitrios Apostolou wrote:
> the attached patch applies after my previous one, and actually
> cancels all runtime gains from it. It doesn't make things worse than
> initially, so it's not *that* bad.
You should really test it on the testcases which lead to
I should note here that specialised hash-tables in pointer-set.c have a
load-factor of at most 25%. Also another very fast hash table I've
studied, dense_hash_map from google's sparse_hash_table, has a load factor
of 50% max.
As I understand it a good hash function gives a perfectly random val
Hello,
the attached patch applies after my previous one, and actually cancels all
runtime gains from it. It doesn't make things worse than initially, so
it's not *that* bad.
While trying to understand var-tracking I deleted the whole shared hash
table concept and some other indirections. It
On Mon, Aug 22, 2011 at 12:53 AM, Artem Shinkarov
wrote:
> Richard
>
> I formalized an approach a little-bit, now it works without target
> hooks, but some polishing is still required. I want you to comment on
> the several important approaches that I use in the patch.
>
> So how does it work.
> 1
Hello,
In c_common_read_pch when gt_pch_restore loads a new pch, the previous
line table (referenced from the global 'line_table') is
garbage-collected and a new one is built.
As the global instance of cpp_reader referenced by the local variable
'pfile' has a pfile->line_table member that referen
Matthias Klose writes:
> 2011-08-22 Matthias Klose
>
> Revert:
> 2011-07-11 Arthur Loiret
> Matthias Klose
> * config/mips/t-linux64 (MULTILIB_DIRNAMES): Set to 'n32 . 64' if
> tm_defines contains MIPS_ABI_DEFAULT ABI_32, to follow the glibc
>
While looking at the multiarch patches, I noticed that a previous change is not
necessary. MULTILIB_DEFAULTS is handled in config/mips/mips.h.
Matthias
gcc/
2011-08-22 Matthias Klose
Revert:
2011-07-11 Arthur Loiret
Matthias Klose
* config/mi
On Mon, Aug 22, 2011 at 12:07 PM, Jakub Jelinek wrote:
> On Mon, Aug 22, 2011 at 11:57:18AM +0200, Richard Guenther wrote:
>> And at some point the idea popped up to just dump this whole re-using
>> mem-attrs. There is at most a single function in RTL but the whole program
>> is available in SSA,
Hello list,
this patch has all of my changes to var-tracking that I believe are worth
keeping. These are all minor changes not touching algorithmic issues, I
lost much time trying to understand what is actually done in
var-tracking.c but I still don't get it. I wish there was some document
de
1 - 100 of 130 matches
Mail list logo
