diff options
author | Richard Biener <rguenther@suse.de> | 2021-02-10 09:06:26 +0100 |
---|---|---|
committer | Richard Biener <rguenther@suse.de> | 2021-02-10 10:09:33 +0100 |
commit | d997565c41a8a5783bf076437208f38d8ea39ced (patch) | |
tree | c3bcc261312d4755f45b186355222bd2322c856b | |
parent | 5da5d8a02c6799e60970fef72ee8c1c3d033a5e5 (diff) | |
download | gcc-d997565c41a8a5783bf076437208f38d8ea39ced.zip gcc-d997565c41a8a5783bf076437208f38d8ea39ced.tar.gz gcc-d997565c41a8a5783bf076437208f38d8ea39ced.tar.bz2 |
tree-optimization/99024 - fix leak in loop vect analysis
When we analyzed a loop as epilogue but later in peeling decide
we're not going to use it then in the DTOR we clear the original
loops ->aux which causes us to leak the main loop vinfo.
Fixed by only clearing aux if it is associated with the vinfo
we're destroying.
2021-02-10 Richard Biener <rguenther@suse.de>
PR tree-optimization/99024
* tree-vect-loop.c (_loop_vec_info::~_loop_vec_info): Only
clear loop->aux if it is associated with the destroyed loop_vinfo.
-rw-r--r-- | gcc/tree-vect-loop.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/gcc/tree-vect-loop.c b/gcc/tree-vect-loop.c index f1f16e3..27845c0 100644 --- a/gcc/tree-vect-loop.c +++ b/gcc/tree-vect-loop.c @@ -927,7 +927,11 @@ _loop_vec_info::~_loop_vec_info () delete scan_map; epilogue_vinfos.release (); - loop->aux = NULL; + /* When we release an epiloge vinfo that we do not intend to use + avoid clearing AUX of the main loop which should continue to + point to the main loop vinfo since otherwise we'll leak that. */ + if (loop->aux == this) + loop->aux = NULL; } /* Return an invariant or register for EXPR and emit necessary |