Hi Jeff, No longer needed. Jakub's patch does the job.
Michael Collison > On Nov 19, 2017, at 3:50 PM, Jeff Law <l...@redhat.com> wrote: > >> On 10/25/2017 11:54 PM, Michael Collison wrote: >> Ping. Original patch posted here: >> >> https://gcc.gnu.org/ml/gcc-patches/2017-10/msg01317.html > Is this still needed or was it addressed by the patch Jakub checked in here: > > > > 2017-11-01 Jakub Jelinek <ja...@redhat.com> > > PR rtl-optimization/82778 > PR rtl-optimization/82597 > * compare-elim.c (struct comparison): Add in_a_setter field. > (find_comparison_dom_walker::before_dom_children): Remove killed > bitmap and df_simulate_find_defs call, instead walk the defs. > Compute last_setter and initialize in_a_setter. Merge definitions > with first initialization for a few variables. > (try_validate_parallel): Use insn_invalid_p instead of > recog_memoized. Return insn rather than just the pattern. > (try_merge_compare): Fix up comment. Don't uselessly test if > in_a is a REG_P. Use cmp->in_a_setter instead of walking UD > chains. > (execute_compare_elim_after_reload): Remove df_chain_add_problem > call.