public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug target/23983] New: the altivec builtins should be marked as pure/const
@ 2005-09-20 17:26 pinskia at gcc dot gnu dot org
  2005-09-20 17:27 ` [Bug target/23983] " pinskia at gcc dot gnu dot org
  0 siblings, 1 reply; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-09-20 17:26 UTC (permalink / raw)
  To: gcc-bugs

#   TMT.5_729 = V_MAY_DEF <TMT.5_727>;
  #   TMT.6_730 = V_MAY_DEF <TMT.6_728>;
  y0_351 = __builtin_altivec_vrsqrtefp (B_350);

Almost all of the altivec builtins could be marked as pure/const and should cause less VOPS to be 
added for altivec code.

-- 
           Summary: the altivec builtins should be marked as pure/const
           Product: gcc
           Version: 4.1.0
            Status: UNCONFIRMED
          Keywords: missed-optimization
          Severity: enhancement
          Priority: P2
         Component: target
        AssignedTo: unassigned at gcc dot gnu dot org
        ReportedBy: pinskia at gcc dot gnu dot org
                CC: gcc-bugs at gcc dot gnu dot org


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
  2005-09-20 17:26 [Bug target/23983] New: the altivec builtins should be marked as pure/const pinskia at gcc dot gnu dot org
@ 2005-09-20 17:27 ` pinskia at gcc dot gnu dot org
  0 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-09-20 17:27 UTC (permalink / raw)
  To: gcc-bugs



-- 
           What    |Removed                     |Added
----------------------------------------------------------------------------
OtherBugsDependingO|                            |14455
              nThis|                            |


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
                   ` (6 preceding siblings ...)
  2008-03-12 20:42 ` pinskia at gcc dot gnu dot org
@ 2010-02-25 20:03 ` meissner at gcc dot gnu dot org
  7 siblings, 0 replies; 10+ messages in thread
From: meissner at gcc dot gnu dot org @ 2010-02-25 20:03 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #8 from meissner at gcc dot gnu dot org  2010-02-25 20:03 -------
Patches checked in on October 15th, 2009 to properly set pure/const.


-- 

meissner at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |meissner at gcc dot gnu dot
                   |                            |org
             Status|ASSIGNED                    |RESOLVED
         Resolution|                            |FIXED


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
                   ` (5 preceding siblings ...)
  2008-03-12 19:17 ` pinskia at gcc dot gnu dot org
@ 2008-03-12 20:42 ` pinskia at gcc dot gnu dot org
  2010-02-25 20:03 ` meissner at gcc dot gnu dot org
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2008-03-12 20:42 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #7 from pinskia at gcc dot gnu dot org  2008-03-12 20:42 -------
So what is happening is we don't model the SAT bit of VSCR on the tree level
(in the builtins) so the saturated builtins cannot be marked as const.  They
are modeled correctly at the RTL level though.


-- 


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
                   ` (4 preceding siblings ...)
  2008-03-11 21:30 ` pinskia at gcc dot gnu dot org
@ 2008-03-12 19:17 ` pinskia at gcc dot gnu dot org
  2008-03-12 20:42 ` pinskia at gcc dot gnu dot org
  2010-02-25 20:03 ` meissner at gcc dot gnu dot org
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2008-03-12 19:17 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #6 from pinskia at gcc dot gnu dot org  2008-03-12 19:17 -------
Note this patch does not fully work as it marks vec_mtvscr as being constant
which it is not.


-- 


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
                   ` (3 preceding siblings ...)
  2006-12-19  8:30 ` pinskia at gcc dot gnu dot org
@ 2008-03-11 21:30 ` pinskia at gcc dot gnu dot org
  2008-03-12 19:17 ` pinskia at gcc dot gnu dot org
                   ` (2 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2008-03-11 21:30 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #5 from pinskia at gcc dot gnu dot org  2008-03-11 21:29 -------
Created an attachment (id=15301)
 --> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=15301&action=view)
Patch which I am testing

This is the patch which I am testing currently for this bug.


-- 


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
                   ` (2 preceding siblings ...)
  2006-11-15  0:39 ` pinskia at gcc dot gnu dot org
@ 2006-12-19  8:30 ` pinskia at gcc dot gnu dot org
  2008-03-11 21:30 ` pinskia at gcc dot gnu dot org
                   ` (3 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-12-19  8:30 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #4 from pinskia at gcc dot gnu dot org  2006-12-19 08:30 -------
My patch for this needs to be rewritten, not because it was wrong but because
it can be cleaned up.


-- 


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
  2005-10-23 23:42 ` pinskia at gcc dot gnu dot org
  2006-04-08  1:29 ` pinskia at gcc dot gnu dot org
@ 2006-11-15  0:39 ` pinskia at gcc dot gnu dot org
  2006-12-19  8:30 ` pinskia at gcc dot gnu dot org
                   ` (4 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-11-15  0:39 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #3 from pinskia at gcc dot gnu dot org  2006-11-15 00:39 -------
I have a start to this patch and it works nicely, I just have to finish up
marking which builtins should be pure and which should be const and which are
neither.


-- 


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
  2005-10-23 23:42 ` pinskia at gcc dot gnu dot org
@ 2006-04-08  1:29 ` pinskia at gcc dot gnu dot org
  2006-11-15  0:39 ` pinskia at gcc dot gnu dot org
                   ` (5 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2006-04-08  1:29 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #2 from pinskia at gcc dot gnu dot org  2006-04-08 01:29 -------
I am going to look into this some more since well PPC/Cell/PS3 is my thing now.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         AssignedTo|unassigned at gcc dot gnu   |pinskia at gcc dot gnu dot
                   |dot org                     |org
             Status|NEW                         |ASSIGNED


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

* [Bug target/23983] the altivec builtins should be marked as pure/const
       [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
@ 2005-10-23 23:42 ` pinskia at gcc dot gnu dot org
  2006-04-08  1:29 ` pinskia at gcc dot gnu dot org
                   ` (6 subsequent siblings)
  7 siblings, 0 replies; 10+ messages in thread
From: pinskia at gcc dot gnu dot org @ 2005-10-23 23:42 UTC (permalink / raw)
  To: gcc-bugs



------- Comment #1 from pinskia at gcc dot gnu dot org  2005-10-23 23:42 -------
Confirmed.


-- 

pinskia at gcc dot gnu dot org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
     Ever Confirmed|0                           |1
   Last reconfirmed|0000-00-00 00:00:00         |2005-10-23 23:42:28
               date|                            |


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


^ permalink raw reply	[flat|nested] 10+ messages in thread

end of thread, other threads:[~2010-02-25 20:03 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2005-09-20 17:26 [Bug target/23983] New: the altivec builtins should be marked as pure/const pinskia at gcc dot gnu dot org
2005-09-20 17:27 ` [Bug target/23983] " pinskia at gcc dot gnu dot org
     [not found] <bug-23983-6528@http.gcc.gnu.org/bugzilla/>
2005-10-23 23:42 ` pinskia at gcc dot gnu dot org
2006-04-08  1:29 ` pinskia at gcc dot gnu dot org
2006-11-15  0:39 ` pinskia at gcc dot gnu dot org
2006-12-19  8:30 ` pinskia at gcc dot gnu dot org
2008-03-11 21:30 ` pinskia at gcc dot gnu dot org
2008-03-12 19:17 ` pinskia at gcc dot gnu dot org
2008-03-12 20:42 ` pinskia at gcc dot gnu dot org
2010-02-25 20:03 ` meissner at gcc dot gnu dot org

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).