[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-07-02 Thread dave at hiauly1 dot hia dot nrc dot ca 


--- Comment #11 from dave at hiauly1 dot hia dot nrc dot ca  2010-07-02 
15:50 ---
Subject: Re:  FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump
pta ESCAPED = { }

On Wed, 30 Jun 2010, rguenth at gcc dot gnu dot org wrote:

 You need the following additional patch.  With that the testcase should pass.

The testsuite now passes.  Tested on hppa2.0w-hp-hpux11.11.  I've attached
the change as tested.

Dave


--- Comment #12 from dave at hiauly1 dot hia dot nrc dot ca  2010-07-02 
15:50 ---
Created an attachment (id=21069)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21069action=view)


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-07-02 Thread rguenth at gcc dot gnu dot org 


--- Comment #13 from rguenth at gcc dot gnu dot org  2010-07-02 15:56 
---
I have committed the middle-end pieces already today, so if you can approve and
commit the pa parts that would be nice.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|rguenth at gcc dot gnu dot  |danglin at gcc dot gnu dot
   |org |org


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-07-02 Thread danglin at gcc dot gnu dot org 


--- Comment #14 from danglin at gcc dot gnu dot org  2010-07-02 20:52 
---
Subject: Bug 43958

Author: danglin
Date: Fri Jul  2 20:51:58 2010
New Revision: 161746

URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=161746
Log:
PR target/43958
* config/pa/pa.c (hppa_gimplify_va_arg_expr): Use pointer arithmetic
for argument alignment.


Modified:
trunk/gcc/ChangeLog
trunk/gcc/config/pa/pa.c


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-07-02 Thread danglin at gcc dot gnu dot org 


--- Comment #15 from danglin at gcc dot gnu dot org  2010-07-02 20:58 
---
Fixed.


-- 

danglin at gcc dot gnu dot org changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution||FIXED


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-06-30 Thread rguenth at gcc dot gnu dot org 


--- Comment #8 from rguenth at gcc dot gnu dot org  2010-06-30 08:27 ---
Can you, instead of

  /* Copied from va-pa.h, but we probably don't need to align to
 word size, since we generate and preserve that invariant.  */
  u = size_int (size  4 ? -8 : -4);
  t = fold_convert (sizetype, t);
  t = build2 (BIT_AND_EXPR, sizetype, t, u);
  t = fold_convert (valist_type, t);

use

  u = build_int_cst (TREE_TYPE (t), (HOST_WIDE_INT)(size  4 ? -8 : -4));
  t = build2 (BIT_AND_EXPR, TREE_TYPE (t), t, u);
  t = fold_convert (valist_type, t);

thus, perform the AND in pointer types?  This is what I am going to do
to replace REALIGN_INDIRECT_REF with and teach PTA about this (so to not
pessimize hppa here).

Mine for now - I'll ping you when infrastructure is ready to deal with
pointer BIT_AND_EXPRs.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 AssignedTo|unassigned at gcc dot gnu   |rguenth at gcc dot gnu dot
   |dot org |org
 Status|NEW |ASSIGNED
   Last reconfirmed|2010-05-02 15:28:17 |2010-06-30 08:27:49
   date||


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-06-30 Thread dave at hiauly1 dot hia dot nrc dot ca 


--- Comment #9 from dave at hiauly1 dot hia dot nrc dot ca  2010-06-30 
14:06 ---
Subject: Re:  FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

 Can you, instead of
 
   /* Copied from va-pa.h, but we probably don't need to align to
  word size, since we generate and preserve that invariant.  */
   u = size_int (size  4 ? -8 : -4);
   t = fold_convert (sizetype, t);
   t = build2 (BIT_AND_EXPR, sizetype, t, u);
   t = fold_convert (valist_type, t);
 
 use
 
   u = build_int_cst (TREE_TYPE (t), (HOST_WIDE_INT)(size  4 ? -8 : -4));
   t = build2 (BIT_AND_EXPR, TREE_TYPE (t), t, u);
   t = fold_convert (valist_type, t);
 
 thus, perform the AND in pointer types?  This is what I am going to do
 to replace REALIGN_INDIRECT_REF with and teach PTA about this (so to not
 pessimize hppa here).

I'll test this change in my next build tonight.

Dave


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-06-30 Thread rguenth at gcc dot gnu dot org 


--- Comment #10 from rguenth at gcc dot gnu dot org  2010-06-30 14:13 
---
Created an attachment (id=21043)
 -- (http://gcc.gnu.org/bugzilla/attachment.cgi?id=21043action=view)
additional patch

You need the following additional patch.  With that the testcase should pass.


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-06-29 Thread danglin at gcc dot gnu dot org 


--- Comment #7 from danglin at gcc dot gnu dot org  2010-06-30 00:24 ---
The re-alignment is necessary, so this is a testsuite issue.  The comment
needs adjustment. 


-- 


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958

[Bug target/43958] FAIL: gcc.dg/ipa/ipa-pta-10.c scan-ipa-dump pta ESCAPED = { }

2010-05-21 Thread rguenth at gcc dot gnu dot org 


--- Comment #6 from rguenth at gcc dot gnu dot org  2010-05-21 10:31 ---
target or testsuite issue.


-- 

rguenth at gcc dot gnu dot org changed:

   What|Removed |Added

 CC||rguenth at gcc dot gnu dot
   ||org
 AssignedTo|rguenth at gcc dot gnu dot  |unassigned at gcc dot gnu
   |org |dot org
 Status|ASSIGNED|NEW
  Component|middle-end  |target


http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43958