On 08/03/14 08:07, Richard Sandiford wrote:
The switch statement in the old code seemed overly cautious.  It's well
established elsewhere that the first operand of an RTX_AUTOINC is the
automodified register.  If anyone wanted to add a new code for which that
wasn't true they should (a) reconsider or (b) go through all RTX_AUTOINCs
as a precaution.


gcc/
        * emit-rtl.c: Include rtl-iter.h.
        (find_auto_inc): Turn from being a for_each_rtx callback to being
        a function that examines each subrtx itself.  Assume the first operand
        to an RTX_AUTOINC is the automodified register.
        (try_split): Update call accordingly.
OK.

It seems like we ought to document somewhere that all autoincrement RTXs increment their first operand. If you can find a goodplace, please add that little doc update as a pre-approved patch.

jeff

Reply via email to