On Mon, Nov 14, 2016 at 11:56 PM, kugan <kugan.vivekanandara...@linaro.org> wrote: > Hi Richard, > > On 08/11/16 23:45, Richard Biener wrote: >> >> On Tue, Nov 8, 2016 at 3:32 AM, kugan <kugan.vivekanandara...@linaro.org> >> wrote: >>> >>> Hi, >>> >>> In tree-ssa-coalesce, register_ssa_partition ) and >>> register_ssa_partition_check have lost their meaning over various commits >>> and now just verifies that ssa_var is indeed a SSA_NAME and not a >>> virtual_operand_p. It is confusing when one look at if for the fist time >>> and >>> would expect more while reading the register_ssa_partition. >>> >>> Attached patch just changes it to verify_ssa_for_coalesce to better >>> reflect >>> what it is doing now. >>> >>> Bootstrap and regression testing is ongoing. Is this OK for trunk if no >>> regressions? >> >> >> Hum, can you retain the inline wrapper please? I find the new name >> verify_ssa_for_coalesce bad as tree-ssa-live.h is something generic, >> not just coalescing related. I'd say a better improvement would be to >> remove >> register_ssa_partition completely. > > Do you like the attached patch which completely removes this.
Yes. Ok if it passes bootstrap/regtest. RIchard. > Thanks, > Kugan > > >> >> Richard. >> >>> Thanks, >>> Kugan >>> >>> >>> >>> gcc/ChangeLog: >>> >>> 2016-11-08 Kugan Vivekanandarajah <kug...@linaro.org> >>> >>> * tree-ssa-coalesce.c (register_default_def): Remove usage of arg >>> map which is not used at all. >>> (create_outofssa_var_map): Use renamed verify_ssa_for_coalesce >>> from >>> register_ssa_partition. >>> * tree-ssa-live.c (verify_ssa_for_coalesce): Renamed >>> register_ssa_partition. >>> (register_ssa_partition_check): Remove. >>> * tree-ssa-live.h (register_ssa_partition): Renamed to >>> verify_ssa_for_coalesce