Remove m_nloops field from loop_versioning
Checks
Context |
Check |
Description |
linaro-tcwg-bot/tcwg_gcc_build--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-arm |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gcc_build--master-aarch64 |
success
|
Testing passed
|
linaro-tcwg-bot/tcwg_gcc_check--master-aarch64 |
success
|
Testing passed
|
Commit Message
This is a small cleanup of loop_versioning where m_nloops
is only used in the constructor so we can remove the whole
field.
Bootstrapped and tested on x86_64-linux-gnu.
gcc/ChangeLog:
* gimple-loop-versioning.cc (loop_versioning): Remove m_nloops field.
(loop_versioning::loop_versioning): Remove initialization of
m_nloops field and move it to be a local variable.
(loop_versioning::analyze_blocks): Fix formating.
---
gcc/gimple-loop-versioning.cc | 7 ++-----
1 file changed, 2 insertions(+), 5 deletions(-)
Comments
On 4/26/24 11:25 PM, Andrew Pinski wrote:
> This is a small cleanup of loop_versioning where m_nloops
> is only used in the constructor so we can remove the whole
> field.
>
> Bootstrapped and tested on x86_64-linux-gnu.
>
> gcc/ChangeLog:
>
> * gimple-loop-versioning.cc (loop_versioning): Remove m_nloops field.
> (loop_versioning::loop_versioning): Remove initialization of
> m_nloops field and move it to be a local variable.
> (loop_versioning::analyze_blocks): Fix formating.
OK. But consider holding off on the commit for a bit to make
cherry-picking to the release branch easier.
jeff
@@ -322,9 +322,6 @@ private:
/* An obstack to use for general allocation. */
obstack m_obstack;
- /* The number of loops in the function. */
- unsigned int m_nloops;
-
/* The total number of loop version conditions we've found. */
unsigned int m_num_conditions;
@@ -525,10 +522,10 @@ loop_versioning::name_prop::value_of_expr (tree val, gimple *)
loop_versioning::loop_versioning (function *fn)
: m_fn (fn),
- m_nloops (number_of_loops (fn)),
m_num_conditions (0),
m_address_table (31)
{
+ unsigned m_nloops = number_of_loops (fn);
bitmap_obstack_initialize (&m_bitmap_obstack);
gcc_obstack_init (&m_obstack);
@@ -1437,7 +1434,7 @@ loop_versioning::analyze_blocks ()
{
linfo.rejected_p = true;
break;
- }
+ }
if (!linfo.rejected_p)
{