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

Jeffrey A. Law <law at redhat dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2011.01.26 17:16:52
                 CC|                            |law at redhat dot com
          Component|bootstrap                   |rtl-optimization
         AssignedTo|unassigned at gcc dot       |law at redhat dot com
                   |gnu.org                     |
     Ever Confirmed|0                           |1

--- Comment #7 from Jeffrey A. Law <law at redhat dot com> 2011-01-26 17:16:52 
UTC ---
More correctly, you can never move any insn between an explicit cc0 setter and
cc0 user on a cc0 target.

Regardless, I think you've nailed this one down.

The comment needs some minor work.   Rather than referring to a PR, we strongly
prefer to describe why the code works the way it does in the code itself.  We
do this so that someone reading this code later can quickly understand its
relevance without having to fire up a browser to read the bug database.

Something as simple as "do not separate a cc0 setter and cc0 user" is
sufficient.  You should also submit the testcase for the regression suite. 
Just name it pr46856.c and put it in gcc/testsuite/gcc.c-torture/compile

I don't see a login for you on gcc.gnu.org, so I'm going to assume you don't
have commit privs.  I'll go ahead and make the cleanups mentioned above and
submit the patch.  You can review the submission so that you know proper form
if you choose to submit patches again in the future.

Reply via email to