This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
Hi Honza,I reverted this patch after it was reported that it resulted in bootstrap compare failure in some targets.
I reproduced it and tracked to a mistake in the patch that introduced it. That is, in propagate_vr_accross_jump_function, I had: if (src_lats->m_value_range.bottom_p ()) return false; which should have been: if (src_lats->m_value_range.bottom_p ()) return dest_lat->set_to_bottom (); I also fixed update_jump_functions_after_inlining as reported in pr78268.I now bootstrapped the patch (lto and normal) on two affected targets aarch64-none-linux-gnu and powerpc64le-unknown-linux-gnu. I also tested it on x86_64-linux-gnu with no new regressions. Is this OK?
Thanks, Kugan gcc/testsuite/ChangeLog: 2016-11-13 Kugan Vivekanandarajah <kuganv@linaro.org> * g++.dg/torture/pr78268.C: New test. gcc/ChangeLog: 2016-11-13 Kugan Vivekanandarajah <kuganv@linaro.org> * ipa-cp.c (ipa_get_jf_pass_through_result): Skip unary expressions. (propagate_vr_accross_jump_function): Handle unary expressions. * ipa-prop.c (ipa_set_jf_unary_pass_through): New. (load_from_param_1): New. (load_from_unmodified_param): Factor common part into load_from_param_1. (load_from_param): New. (compute_complex_assign_jump_func): Handle unary expressions. (update_jump_functions_after_inlining): Likewise. (ipa_write_jump_function): Likewise. (ipa_read_jump_function): Likewise.
Attachment:
p.txt
Description: Text document
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |