[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-26 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 Alexander Monakov changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|---

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-24 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #12 from Alexander Monakov --- Thanks. It's probably nicer to deduplicate computation of required padding to a common header (libcpp/internal.h), I'll send a patch to that effect.

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-24 Thread dcb314 at hotmail dot com via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #11 from David Binderman --- I confirm that the fix works for me. On a more subtle note, if two functions are strongly related, would it be wise to have them in the same file with some comments and maybe even some asserts to ensure

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-23 Thread cvs-commit at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #10 from GCC Commits --- The master branch has been updated by Alexander Monakov : https://gcc.gnu.org/g:b2c1d7c4573d3b938f44b3bda202adeb292b1cbc commit r15-3121-gb2c1d7c4573d3b938f44b3bda202adeb292b1cbc Author: Alexander Monakov

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #9 from Alexander Monakov --- Okay, if you take the addition and the branch from the inlined variant: addl %eax, %edx je .L3 and add a 'test' instruction: addl %eax, %edx test %edx, %edx je .L3 then Valgrind doesn't complain. So

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #8 from Alexander Monakov --- Thanks for the reference, but it doesn't help. Something more subtle is going on, because placing the shift-add combo in a separate function makes Valgrind properly compute known bits even without the ma

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread mark at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #7 from Mark Wielaard --- You could try --expensive-definedness-checks=yes --expensive-definedness-checks= [default: auto] Controls whether Memcheck should employ more precise but also more expensive (ti

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 --- Comment #6 from Alexander Monakov --- As for Valgrind false positive, it handles this SSSE3 code really well and misses the key point by a very narrow margin. We have found = m1 + (m2 << 16); where both m1 and m2 hold 16-bit masks from p

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread amonakov at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 Alexander Monakov changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Ever confirmed|0

[Bug preprocessor/116458] [15 regression] New valgrind error in search_line_ssse3

2024-08-22 Thread pinskia at gcc dot gnu.org via Gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=116458 Andrew Pinski changed: What|Removed |Added Target Milestone|--- |15.0 Host|