Ajit Agarwal <aagar...@linux.ibm.com> writes: > Hello Alex/Richard: > > Renaming of generic code is done to make target independent > and target dependent code to support multiple targets. > > Target independent code is the Generic code with pure virtual function > to interface betwwen target independent and dependent code. > > Target dependent code is the implementation of pure virtual function for > aarch64 target and the call to target independent code. > > Bootstrapped and regtested on aarch64-linux-gnu. > > Thanks & Regards > Ajit > > aarch64: Further renaming of generic code > > Renaming of generic code is done to make target independent > and target dependent code to support multiple targets. > > Target independent code is the Generic code with pure virtual function > to interface betwwen target independent and dependent code. > > Target dependent code is the implementation of pure virtual function for > aarch64 target and the call to target independent code. > > 2024-05-20 Ajit Kumar Agarwal <aagar...@linux.ibm.com> > > gcc/ChangeLog: > > * config/aarch64/aarch64-ldp-fusion.cc: Renaming of generic code
* config/aarch64/aarch64-ldp-fusion.cc: Rename generic parts of code to avoid "ldp" and "stp". > --- > gcc/config/aarch64/aarch64-ldp-fusion.cc | 55 ++++++++++++------------ > 1 file changed, 28 insertions(+), 27 deletions(-) > > [...] > @@ -1126,7 +1127,7 @@ struct stp_change_builder > > bool done () const { return m_state == state::DONE; } > > - stp_change_builder (insn_info *insns[2], > + store_change_builder (insn_info *insns[2], > insn_info *repurpose, > insn_info *dest) Please reindent the parameters for the new longer name. > : m_state (state::FIRST), m_insns { insns[0], insns[1] }, > [...] > @@ -1916,7 +1917,7 @@ fixup_debug_uses (obstack_watermark &attempt, > // BASE gives the chosen base candidate for the pair and MOVE_RANGE is > // a singleton range which says where to place the pair. > bool > -ldp_bb_info::fuse_pair (bool load_p, > +pair_fusion_bb_info::fuse_pair (bool load_p, > unsigned access_size, > int writeback, > insn_info *i1, insn_info *i2, Same here. > @@ -2687,7 +2688,7 @@ pair_fusion::get_viable_bases (insn_info *insns[2], > // ACCESS_SIZE gives the (common) size of a single access, LOAD_P is true > // if the accesses are both loads, otherwise they are both stores. > bool > -ldp_bb_info::try_fuse_pair (bool load_p, unsigned access_size, > +pair_fusion_bb_info::try_fuse_pair (bool load_p, unsigned access_size, > insn_info *i1, insn_info *i2) > { > if (dump_file) And here. OK with those changes, thanks. Richard