[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-06-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #14 from Martin Liška --- (In reply to Jan Hubicka from comment #13) > Great, do you have patch? > Honza Yes, I've suggested a patch here: https://bugzilla.mozilla.org/show_bug.cgi?id=1167119 Let's see if it will be accepted by Mozi

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-06-09 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #13 from Jan Hubicka --- Great, do you have patch? Honza

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-06-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Martin Liška changed: What|Removed |Added Status|UNCONFIRMED |RESOLVED Resolution|---

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-06-09 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #11 from Martin Liška --- Fixing all sanitizer issues in Firefox helped me to successfully built FF with LTO. The binary does not segfault :) Martin

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-21 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #10 from Martin Liška --- Firefox developers just fixed first half of problem seen by null sanitizer and I would still wait for fixing the rest: https://bugzilla.mozilla.org/show_bug.cgi?id=1167119. Looks fixed issues are not suffici

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread law at redhat dot com
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #9 from Jeffrey A. Law --- We don't thread because we don't lower UBSAN_NULL to actual conditionals until the end of the gimple/ssa pipeline. ie, in DOM2 we have: int test::foo(int&) (struct test * const this, int & b) { int _5;

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Jan Hubicka changed: What|Removed |Added CC||jason at redhat dot com --- Comment #8 fro

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread hubicka at ucw dot cz
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #7 from Jan Hubicka --- > According to -fsanitize=null, there are many places in Firefox that produce > undefined behavior in followin way: > > https://bugzilla.mozilla.org/show_bug.cgi?id=1165904 > > One common example: > > st

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #6 from Martin Liška --- According to -fsanitize=null, there are many places in Firefox that produce undefined behavior in followin way: https://bugzilla.mozilla.org/show_bug.cgi?id=1165904 One common example: static size_t off

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Richard Biener changed: What|Removed |Added Target Milestone|4.0.1 |6.0

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-18 Thread rguenth at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Richard Biener changed: What|Removed |Added Target Milestone|--- |4.0.1

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-16 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Jan Hubicka changed: What|Removed |Added CC||law at redhat dot com --- Comment #5 from

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-16 Thread mpolacek at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Marek Polacek changed: What|Removed |Added CC||mpolacek at gcc dot gnu.org --- Comment

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-15 Thread hubicka at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 Jan Hubicka changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment #3

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-15 Thread marxin at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #2 from Martin Liška --- (In reply to Andrew Pinski from comment #1) > This really sounds like a bug in Firefox. This argument is not valid to be > null. Hmm, I suspect there should be an undefined sanitizer that should be > added to

[Bug tree-optimization/66163] [6 Regression] Not working Firefox built with LTO

2015-05-15 Thread pinskia at gcc dot gnu.org
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=66163 --- Comment #1 from Andrew Pinski --- This really sounds like a bug in Firefox. This argument is not valid to be null. Hmm, I suspect there should be an undefined sanitizer that should be added to catch this case if not already there.