This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [PATCH 1/2][GCC] Refactor noce_try_store_flag_constants


On 05/23/2016 03:58 PM, Mikhail Maltsev wrote:
This patch refactors 'noce_try_store_flag_constants' a bit to make it easier to
reason about.

The function contains two series of conditions, and each branch of the second
series corresponds to one or two branches of the first series. The patch
introduces a new enumeration strategy_t instead and uses it to select the
correct branch. Also, ISTM that the last 'else' branch is unreachable.

Bootstrapped and regtested on x86_64-linux. OK for trunk?

Please send patches as text/plain or text/x-patch attachments so they can be read and quoted.

+      bool can_reverse
+ = (reversed_comparison_code (if_info->cond, if_info->jump) != UNKNOWN);

Parens not needed anymore if the line is split this way. Otherwise OK. Maybe the test for common without ST_ADD_FLAG could be moved up a little to avoid unnecessary operations.


Bernd


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]