Hello,
May I have a review from the DWARF, the ARM and RS6000 maintainers for
comments/approval ?
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01613.html
Needed to fix the powerpc-spe bootstrap referenced in bugzilla #57389
Many Thanks
Christian
Christian Bruel wrote:
>> However I feel a li
> I experimented with something like this a while ago. The goal was to
> have a -f{big,little}-endian switch that changes the default byteorder,
> and also to add endianness attributes. It didn't go very far, but I
> could compile some simple testcases and it seemed to behave as expected
> for thos
[Forwarding from http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57578 , as
suggested]
Hi!
The build of powerpc spe on Linux aborts like this:
[...]
/«PKGBUILDDIR»/build/./gcc/xgcc -B/«PKGBUILDDIR»/build/./gcc/
-B/usr/lib/gcc-snapshot/powerpc-linux-gnuspe/bin/
-B/usr/lib/gcc-snapshot/powerpc-lin
On 21/03/13 14:20, Tom Tromey wrote:
>> "Phil" == Phil Muldoon writes:
>
> Phil> 2013-03-21 Phil Muldoon
> Phil> PR gdb/15195
>
> I think this should use a full URL.
> Phil> def to_string (self):
>
> Why doesn't the 'children' method also need a fix?
>
> Also, Phil and I d
On 9 June 2013 05:31, Xinliang David Li wrote:
> Guard also with L_IPO_COMP_MODE as this is lipo specific.
Sounds like this is the LIPO incarnation of http://gcc.gnu.org/PR31537
See the patch at http://gcc.gnu.org/PR31537#c9 for not adding the
alias in the first place (back then, honza may have
ping, CCing middle-end maintainers for review.
On 31 May 2013 10:13, Chung-Lin Tang wrote:
> On 13/5/15 8:12 PM, Richard Sandiford wrote:
>> Chung-Lin Tang writes:
>>> On 13/5/10 6:37 PM, Richard Sandiford wrote:
Chung-Lin Tang writes:
> +case UNSPEC:
> + /* Reach for a co
> On 9 June 2013 05:31, Xinliang David Li wrote:
> > Guard also with L_IPO_COMP_MODE as this is lipo specific.
>
> Sounds like this is the LIPO incarnation of http://gcc.gnu.org/PR31537
>
> See the patch at http://gcc.gnu.org/PR31537#c9 for not adding the
> alias in the first place (back then, h
> Ok, thanks.
>
Checked into MT: http://gcc.gnu.org/ml/gcc-cvs/2013-06/msg00376.html
I think we also should port core-avx2 into 4.8.x
Thanks, K
Hi,
tested x86_64-linux, committed to mainline.
Thanks,
Paolo.
2013-06-11 Paolo Carlini
PR libstdc++/56019
* include/c_global/cstddef (max_align_t): Add to namespace std.
* include/c_std/cstddef: Likewise.
* testsuite/18_support/max_a
Hello Mikael,
Mikael Morin wrote:
Le 03/06/2013 16:06, Tobias Burnus a écrit :
diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c
index b2e8fdc..655d3c1 100644
--- a/gcc/fortran/resolve.c
+++ b/gcc/fortran/resolve.c
@@ -9293,8 +9293,12 @@ get_temp_from_expr (gfc_expr *e, gfc_namespace *
On Tue, Jun 11, 2013 at 01:42:47PM +0400, Kirill Yukhin wrote:
> > Ok, thanks.
> >
> Checked into MT: http://gcc.gnu.org/ml/gcc-cvs/2013-06/msg00376.html
> I think we also should port core-avx2 into 4.8.x
If you mean the documentation of it plus atom MOVBE listing, yes.
Patch preapproved.
Le 11/06/2013 12:00, Tobias Burnus a écrit :
> Hello Mikael,
>
> Mikael Morin wrote:
>> Le 03/06/2013 16:06, Tobias Burnus a écrit :
>>> diff --git a/gcc/fortran/resolve.c b/gcc/fortran/resolve.c
>>> index b2e8fdc..655d3c1 100644
>>> --- a/gcc/fortran/resolve.c
>>> +++ b/gcc/fortran/resolve.c
>>>
Hi!
As discussed on IRC, decl_binds_to_current_def_p doesn't handle
some cases properly, e.g. if a DECL_COMMON decl has hidden visibility,
then it returns true even without consulting linker plugin resolution,
even when the common might bind to a real definition of the var in some
other TU of the
Hi,
The dg-extract-result.sh script is not tolerant of spaces in test variants.
Consider the following site.exp fragment:
set target_list [
list "arm-eabi-aem{-marm -march=armv7-a -mfpu=vfpv3-d16
-mfloat-abi=softfp}" \
"arm-eabi-aem{-marm -march=armv8-a -mfpu=crypto-neon-fp-armv8
On Tue, Jun 11, 2013 at 01:12:50PM +0100, Marcus Shawcroft wrote:
> Hi,
>
> The dg-extract-result.sh script is not tolerant of spaces in test variants.
>
> Consider the following site.exp fragment:
>
>
> set target_list [
> list "arm-eabi-aem{-marm -march=armv7-a -mfpu=vfpv3-d16
> -mfloat
On 11/06/13 13:55, Jakub Jelinek wrote:
On Tue, Jun 11, 2013 at 01:12:50PM +0100, Marcus Shawcroft wrote:
Hi,
The dg-extract-result.sh script is not tolerant of spaces in test variants.
Consider the following site.exp fragment:
set target_list [
list "arm-eabi-aem{-marm -march=armv7-a
On Tue, Jun 11, 2013 at 02:07:50PM +0100, Marcus Shawcroft wrote:
> On 11/06/13 13:55, Jakub Jelinek wrote:
> >On Tue, Jun 11, 2013 at 01:12:50PM +0100, Marcus Shawcroft wrote:
> >>Hi,
> >>
> >>The dg-extract-result.sh script is not tolerant of spaces in test variants.
> >>
> >>Consider the followi
On 11/06/13 14:12, Jakub Jelinek wrote:
On Tue, Jun 11, 2013 at 02:07:50PM +0100, Marcus Shawcroft wrote:
On 11/06/13 13:55, Jakub Jelinek wrote:
On Tue, Jun 11, 2013 at 01:12:50PM +0100, Marcus Shawcroft wrote:
Hi,
The dg-extract-result.sh script is not tolerant of spaces in test variants.
>> I think I previously put constraint_info in lang_decl_min, right next
>> to template_info no less. It was easy to manage there, and initialized
>> as part of build_template_decl. But this obviously doesn't work for
>> partial specializations unless they get template_decls.
>
>
> Right. And we w
Hi,
This patch split the simd aarch64_combine pattern.
This passes the full regression test suite in aarch64-elf.
OK for trunk?
-
Thanks
Sofiane
aarch64-split-simd-combine.diff
Description: Binary data
Hi,
This patch fixes a bug in the move_lo_quad_ pattern.
The pattern, shown below, issues a scalar MOV instruction for vector modes:
(define_insn "move_lo_quad_"
[(set (match_operand:VQ 0 "register_operand" "=w,w,w")
(vec_concat:VQ
(match_operand: 1 "register_operand" "w,r,r"
Hi DJ,
> "umul" is used when the insn takes two operands in A and X, and computes a
> value into AX.
> "macax" is used when the insn pattern itself reads from two virtual registers
> and writes to a
> virtual register (i.e. V = V op V), but clobbers AX anx BC in the process.
Thanks for the cl
On 06/11/2013 09:45 AM, Andrew Sutton wrote:
After investigating, neither call_expr nor resolve_nondeduced_context
do what I need. I need a resolution of a call expression that does not
return overload sets, especially in the case where the initial call
expression is already dependent.
Does thi
Le 06/06/2013 17:10, Tobias Burnus a écrit :
> The attached test case failed with an ICE for function result variables
> as in that case the function decl was used.
>
> Build and regtested on x86-64-gnu-linux.
> OK for the trunk?
>
Yes, thanks
Mikael
On Mon, 10 Jun 2013 09:02:24 -0700
Cesar Philippidis wrote:
> On 6/10/13 8:32 AM, Richard Earnshaw wrote:
> > On 07/06/13 17:50, Cesar Philippidis wrote:
> >> On 6/6/13 9:00 AM, Richard Earnshaw wrote:
> >>> The pipeline offset is 4 for Thumb2 as well. So at the very
> >>> least you need to expl
>> After investigating, neither call_expr nor resolve_nondeduced_context
>> do what I need. I need a resolution of a call expression that does not
>> return overload sets, especially in the case where the initial call
>> expression is already dependent.
>
>
> Does this have to do with restrictions
On Tue, Jun 11, 2013 at 4:36 AM, Roland Stigge wrote:
> [Forwarding from http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57578 , as
> suggested]
>
> Hi!
>
> The build of powerpc spe on Linux aborts like this:
>
> [...]
> /«PKGBUILDDIR»/build/./gcc/xgcc -B/«PKGBUILDDIR»/build/./gcc/
> -B/usr/lib/gc
Usage of "debug_prefix_maps" is hidden behind an API, with the only
direct uses of the ptr being within final.c. Indeed the declaration of
its type is local to that file.
Hence it can be made static.
Successfully bootstrapped on x86_64-unknown-linux-gnu
OK for trunk?
2013-06-11 David Malcolm
On 06/11/2013 10:49 AM, Andrew Sutton wrote:
After investigating, neither call_expr nor resolve_nondeduced_context
do what I need. I need a resolution of a call expression that does not
return overload sets, especially in the case where the initial call
expression is already dependent.
Does thi
On Tue, 11 Jun 2013 15:42:24 +0100
Julian Brown wrote:
> Ahh -- so actually, maybe the right thing to do is use this last style
> of thunk for v6-m, the same as v7-m -- is there any reason that
> wouldn't work? (I wrote this patch to start with, but I don't remember
> if I considered that possibi
> And you need to do this even when the call is type-dependent?
Yes. Especially when the call is type-dependent. That's how I generate
the constraint sets, which are used to determine the most constrained
template during overload resolution.
On 11/06/13 14:44, Sofiane Naci wrote:
Hi,
This patch split the simd aarch64_combine pattern.
This passes the full regression test suite in aarch64-elf.
OK for trunk?
-
Thanks
Sofiane
OK
/Marcus
Both function.c and varasm.c contain their own copies of this
declaration:
static GTY(()) rtx initial_trampoline;
Only the one in varasm.c is used; the one in function.c appears to be a
missing removal from this move:
2004-05-13 Diego Novillo
Merge from tree-ssa-20020619-branch.
Thanks!
> ; end of mulqi macro"
> +;; [(set_attr "valloc" "umul")]
I think this one needs to be macax also, since the constraints have
"v" in them but the opcodes use ax.
Other than that, it's OK.
On 06/11/13 09:16, David Malcolm wrote:
2013-06-11 David Malcolm
* function.c (initial_trampoline): Remove stray copy; see
varasm.c for the real copy.
This is fine. However, I'd just make the ChangeLog
* function.c (initial_trampoline): Remove stray copy.
Jeff
On 06/10/13 15:36, Oleg Endo wrote:
Hello,
Any comments on that one?
Original message containing the attachment is here:
http://gcc.gnu.org/ml/gcc-patches/2013-05/msg01086.html
On Mon, 2013-05-20 at 18:06 +0200, Oleg Endo wrote:
Hi,
I've noticed that testsuite/gcc.dg has some ISR related te
On 06/07/13 11:36, David Malcolm wrote:
"interesting_blocks" is only used inside tree-into-ssa.c, so it can be
made static.
Successfully bootstrapped on x86_64-unknown-linux-gnu (using
gcc-4.7.2-2.fc17.x86_64).
OK for trunk?
2013-06-07 David Malcolm
* tree-into-ssa.c (interesting_b
On 06/07/13 11:40, David Malcolm wrote:
The array "unknown" within tree-object-size.c is only ever read from,
not written to, so it can be marked as const.
Successfully bootstrapped on x86_64-unknown-linux-gnu (using
gcc-4.7.2-2.fc17.x86_64).
OK for trunk?
2013-06-07 David Malcolm
Here is a simple one. When processing CONST_DECLs after an error, we can ICE.
This avoid the ICE.
Ok?
2013-06-11 Mike Stump
* init.c (constant_value_1): Protect CONST_DECLs better in the
face of errors.
diff --git a/gcc/cp/init.c b/gcc/cp/init.c
index 44e558e..133a162 10
> -Original Message-
> From: Maciej W. Rozycki [mailto:ma...@codesourcery.com]
> Sent: Monday, June 10, 2013 8:50 PM
> To: Steve Ellcey; Moore, Catherine
> Cc: Richard Sandiford; gcc-patches@gcc.gnu.org
> Subject: Re: [patch, mips] Micromips delay slot fix
>
> On Tue, 11 Jun 2013, Steve
On 06/11/2013 07:57 AM, David Malcolm wrote:
> Usage of "debug_prefix_maps" is hidden behind an API, with the only
> direct uses of the ptr being within final.c. Indeed the declaration of
> its type is local to that file.
>
> Hence it can be made static.
>
> Successfully bootstrapped on x86_64-u
On 06/11/2013 06:27 PM, Mike Stump wrote:
Here is a simple one. When processing CONST_DECLs after an error, we can ICE.
This avoid the ICE.
No testcase?
Paolo.
On Sat, Apr 27, 2013 at 08:17:35PM +0200, Jakub Jelinek wrote:
> One way we could implement the SIMD private/lastprivate/reduction
> vars and for Cilk+ also firstprivate ones might be:
> - query the target what the maximum possible vectorization factor for the
> loop is (and min that with simdlen
On 06/11/2013 04:31 AM, Jakub Jelinek wrote:
> +bool
> +default_binds_to_current_def_p_1 (const_tree exp, int shlib)
> +{
...
> +}
I'm having trouble picking out what's supposed to be different between this
function and default_binds_local_p_1. It appears to be the same set of tests,
just rearran
On Tue, Jun 11, 2013 at 09:56:28AM -0700, Richard Henderson wrote:
> On 06/11/2013 04:31 AM, Jakub Jelinek wrote:
> > +bool
> > +default_binds_to_current_def_p_1 (const_tree exp, int shlib)
> > +{
> ...
> > +}
>
> I'm having trouble picking out what's supposed to be different between this
> functi
On Tue, 2013-06-11 at 09:56 -0600, Jeff Law wrote:
> On 06/11/13 09:16, David Malcolm wrote:
> >
> > 2013-06-11 David Malcolm
> >
> > * function.c (initial_trampoline): Remove stray copy; see
> > varasm.c for the real copy.
> This is fine. However, I'd just make the ChangeLog
>
>
On Tue, 2013-06-11 at 09:34 -0700, Richard Henderson wrote:
> On 06/11/2013 07:57 AM, David Malcolm wrote:
> > Usage of "debug_prefix_maps" is hidden behind an API, with the only
> > direct uses of the ptr being within final.c. Indeed the declaration of
> > its type is local to that file.
> >
> >
On 06/11/2013 10:03 AM, Jakub Jelinek wrote:
> On Tue, Jun 11, 2013 at 09:56:28AM -0700, Richard Henderson wrote:
>> I'm having trouble picking out what's supposed to be different between this
>> function and default_binds_local_p_1. It appears to be the same set of
>> tests,
>> just rearranged i
"Moore, Catherine" writes:
> I'm testing a slightly different patch from the one that Steve posted:
>
> Index: mips.md
> ===
> --- mips.md (revision 199648)
> +++ mips.md (working copy)
> @@ -703,8 +703,13 @@
>
> ;; Is it a s
On 06/11/2013 11:09 AM, Andrew Sutton wrote:
And you need to do this even when the call is type-dependent?
Yes. Especially when the call is type-dependent. That's how I generate
the constraint sets, which are used to determine the most constrained
template during overload resolution.
Ah, that
On 06/11/2013 12:39 PM, Paolo Carlini wrote:
On 06/11/2013 06:27 PM, Mike Stump wrote:
Here is a simple one. When processing CONST_DECLs after an error, we
can ICE. This avoid the ICE.
No testcase?
Yep; the patch is fine, but needs a testcase.
Jason
On Tue, Jun 11, 2013 at 10:18:17AM -0700, Richard Henderson wrote:
> > 2) DECL_COMMON with non-default visibility - the old version would return
> >true, now it returns false unless linker plugin tells us the current
> >common was used
> >
> > Or do you think we should just do what we did
On 06/11/2013 11:20 AM, Jakub Jelinek wrote:
> 2013-06-11 Jakub Jelinek
>
> PR target/56564
> * varasm.c (decl_binds_to_current_def_p): Call binds_local_p
> target hook even for !TREE_PUBLIC decls. If no resolution info
> is available, return false for common and extern
On 06/09/13 10:25, Marc Glisse wrote:
Hello,
this patch removes some self-assignments. I don't know if this is the
best way, but it passes a bootstrap and the testsuite on x86_64-linux-gnu.
2013-06-10 Marc Glisse
PR tree-optimization/57361
gcc/
* tree-ssa-dse.c (dse_possible_dead_
> 2013-06-11 Jakub Jelinek
>
> PR target/56564
> * varasm.c (decl_binds_to_current_def_p): Call binds_local_p
> target hook even for !TREE_PUBLIC decls. If no resolution info
> is available, return false for common and external decls.
>
> --- gcc/varasm.c.jj 2013-06-
On Sat, Jun 08, 2013 at 08:22:33PM +0200, Jakub Jelinek wrote:
> > >+ tt = fold_build2 (EQ_EXPR, boolean_type_node, op1,
> > >+ integer_minus_one_node);
> >
> > Don't we usually try to have both operands of a comparison of the
> > same type?
>
> Not just usually, it really has to be
On Tue, Jun 11, 2013 at 08:39:07PM +0200, Jan Hubicka wrote:
> > @@ -6802,10 +6802,20 @@ decl_binds_to_current_def_p (tree decl)
> > return resolution_to_local_definition_p (node->symbol.resolution);
> > }
> >/* Otherwise we have to assume the worst for DECL_WEAK (hidden weaks
> > -
Fixes a libstdc++v3 build failure. Committed.
* config/rl78/rl78.c (TARGET_UNWIND_WORD_MODE): Define.
(rl78_unwind_word_mode): New.
Index: config/rl78/rl78.c
===
--- config/rl78/rl78.c (revision 199963)
+++ config
On 06/09/13 13:43, Marc Glisse wrote:
Hello,
just adapting yet another function so it also works with vectors.
It seemed convenient to add a new macro. The name sucks (it doesn't
match the semantics of INTEGRAL_TYPE_P), but I didn't want to name it
INTEGER_SCALAR_OR_VECTOR_CONSTANT_P and didn't
> On Tue, Jun 11, 2013 at 08:39:07PM +0200, Jan Hubicka wrote:
> > > @@ -6802,10 +6802,20 @@ decl_binds_to_current_def_p (tree decl)
> > > return resolution_to_local_definition_p (node->symbol.resolution);
> > > }
> > >/* Otherwise we have to assume the worst for DECL_WEAK (hidden weaks
Hello,
couple comments (not a true review)
On Tue, 11 Jun 2013, Marek Polacek wrote:
+tree
+ubsan_instrument_division (location_t loc, tree op0, tree op1)
+{
+ tree t, tt;
+ tree type0 = TREE_TYPE (op0);
+ tree type1 = TREE_TYPE (op1);
Can the 2 types be different? I thought divisions had
On Tue, 11 Jun 2013, Jeff Law wrote:
On 06/09/13 10:25, Marc Glisse wrote:
Hello,
this patch removes some self-assignments. I don't know if this is the
best way, but it passes a bootstrap and the testsuite on x86_64-linux-gnu.
2013-06-10 Marc Glisse
PR tree-optimization/57361
gcc/
On Tue, 11 Jun 2013, Jeff Law wrote:
On 06/09/13 13:43, Marc Glisse wrote:
Hello,
just adapting yet another function so it also works with vectors.
It seemed convenient to add a new macro. The name sucks (it doesn't
match the semantics of INTEGRAL_TYPE_P), but I didn't want to name it
INTEGER
Hi!
On Tue, Jun 11, 2013 at 09:14:36PM +0200, Marc Glisse wrote:
> Hello,
>
> couple comments (not a true review)
Thanks anyway ;).
> On Tue, 11 Jun 2013, Marek Polacek wrote:
>
> >+tree
> >+ubsan_instrument_division (location_t loc, tree op0, tree op1)
> >+{
> >+ tree t, tt;
> >+ tree type0
On Tue, Jun 11, 2013 at 09:30:29PM +0200, Marc Glisse wrote:
> >I'd be curious how often this triggers in GCC itself as well.
>
> Do you know a convenient way to test that?
Perhaps you could put in the
if (gimple_assign_rhs_code (stmt) == TREE_CODE (gimple_assign_lhs (stmt))
&& operand_equal
Hi,
this issue, which originally was about spelling out in the error message
the ambiguous candidates, is by and large fixed. Only, we produce the
whole diagnostic twice for the testcase at issue: first in
cp_parser_lookup_name and then again in
cp_parser_nested_name_specifier_opt. The below
On Tue, Jun 11, 2013 at 09:44:40PM +0200, Marek Polacek wrote:
> > >+ tree type0 = TREE_TYPE (op0);
> > >+ tree type1 = TREE_TYPE (op1);
> >
> > Can the 2 types be different? I thought divisions had homogeneous
> > arguments, and the instrumentation was done late enough to avoid any
> > potentia
On Tue, Jun 11, 2013 at 10:09:00PM +0200, Jakub Jelinek wrote:
> On Tue, Jun 11, 2013 at 09:44:40PM +0200, Marek Polacek wrote:
> > > >+ tree type0 = TREE_TYPE (op0);
> > > >+ tree type1 = TREE_TYPE (op1);
> > >
> > > Can the 2 types be different? I thought divisions had homogeneous
> > > argume
On Tue, Jun 11, 2013 at 10:20:24PM +0200, Marek Polacek wrote:
> On Tue, Jun 11, 2013 at 10:09:00PM +0200, Jakub Jelinek wrote:
> > On Tue, Jun 11, 2013 at 09:44:40PM +0200, Marek Polacek wrote:
> > > > >+ tree type0 = TREE_TYPE (op0);
> > > > >+ tree type1 = TREE_TYPE (op1);
> > > >
> > > > Can
On Tue, Jun 11, 2013 at 10:33:25PM +0200, Jakub Jelinek wrote:
> That means you probably should move the function call down in
> cp_build_binary_op (resp. C counterpart), after the arguments are converted
> to result_type?
Ok, certainly. Seems the arguments are converted here:
if (! converted)
On Tue, Jun 11, 2013 at 10:40:12PM +0200, Marek Polacek wrote:
> On Tue, Jun 11, 2013 at 10:33:25PM +0200, Jakub Jelinek wrote:
> > That means you probably should move the function call down in
> > cp_build_binary_op (resp. C counterpart), after the arguments are converted
> > to result_type?
>
>
On Tue, Jun 11, 2013 at 10:44:12PM +0200, Jakub Jelinek wrote:
> There is another thing to solve BTW, op0 and/or op1 might have side-effects,
> if you are going to evaluate them more than once, they need to be surrounded
> into cp_save_expr resp. c_save_expr.
I see. Thanks for the notice.
This patch fixes a testsuite failure when -fdebug-types-section is
turned on, where the order of type stub DIEs is dependent on hash
table traversal order.
Backported from trunk at r199927. Bootstrapped, tested, and committed.
-cary
2013-06-10 Cary Coutant
gcc/
* dwarf2out.c (hash_e
Hi,
weakref is a static variable that is an alias of external symbol. Either by GCC
or by GAS it is translated to references to the target and depending on whether
target is used or not it is declared as weak or non-weak symbol.
Now the implementation in c-common.c sets DECL_EXTERNAL flag when bu
Hello Richard,
> > How much other changes will be currently accepted here? There is other
> > stuff which I want to prepare and submit here, e.g.:
> > 1. disable use of dmult and ddiv (ABI n32).
> > 2. use trunc.w.s instead of cvt.w.s (to get single float working for
> > normal range calculations;
I committed the following patch as obvious, after making sure it builds and has
no regressions. I was looking at driver-rs6000.c today, and I noticed that it
was returning an address to an auto buffer for processing -mcpu=native.
2013-06-11 Michael Meissner
PR target/57589
* c
On Tue, Jun 11, 2013 at 07:25:36PM -0400, Michael Meissner wrote:
> I committed the following patch as obvious, after making sure it builds and
> has
> no regressions. I was looking at driver-rs6000.c today, and I noticed that it
> was returning an address to an auto buffer for processing -mcpu=n
On Wed, Jun 12, 2013 at 01:34:59AM +0200, Jakub Jelinek wrote:
> On Tue, Jun 11, 2013 at 07:25:36PM -0400, Michael Meissner wrote:
> > I committed the following patch as obvious, after making sure it builds and
> > has
> > no regressions. I was looking at driver-rs6000.c today, and I noticed that
I needed to rework the sync.md so that it would work correctly with no
optimization (using SUBREG's at -O0 did not give us the even registers for
holding PTImode values, so I created a PTImode temporary in load_lockedti and
store_conditionalti, which is normally optimized out.
[gcc]
2013-06-11 Mi
The following patch fixes PR57134 by
a) excluding bitfield expansion when EXPAND_MEMORY, and
b) passing down the EXPAND_MEMORY modifier in a couple of places where
this does not currently happen on recursive calls to expand_expr().
(a) has precedent elsewhere in expr.c, eg. see expand_expr_real_1
>> +;; [(set_attr "valloc" "umul")]
>I think this one needs to be macax also, since the constraints have "v" in
>them but the opcodes use ax.
> Other than that, it's OK.
Committed with above change.
Thanks for the quick review.
Regards,
Kaushik
On 06/07/13 03:14, Richard Biener wrote:
+/* Given SSA_NAMEs NAME1 and NAME2, return true if they are candidates for
+ coalescing together, false otherwise.
+
+ This must stay consistent with the code in tree-ssa-live.c which
+ sets up base values in the var map. */
+
+bool
+gimple_can_
On Tue, Jun 11, 2013 at 04:58:07PM +, Joseph S. Myers wrote:
> It's preferred to put the hook documentation in the doc string in
> target.def, so tm.texi.in then only contains the @hook line indicating
> where the documentation will go in tm.texi.
Ah, thanks. Revised patch with testcase.
g
Hi!
The darwin and PE binds_local_p targhooks signal that even for -fpic
symbols can't be normally interposed, thus the testcases aren't
usable on these targets, because they rely on possible symbol interposition.
Fixed thusly, committed as obvious.
2013-06-12 Jakub Jelinek
PR target
85 matches
Mail list logo
