[valgrind] [Bug 476025] Vbit expected test results for Iop_CmpGT64Ux2 are wrong.

2024-04-11 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=476025 --- Comment #12 from Eyal --- But this one is not merged: https://bugsfiles.kde.org/attachment.cgi?id=162566 Should it be merged? -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 474160] If errors-for-leak-kinds is specified, exit-on-first-error should only exit on one of the listed errors.

2024-03-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=474160 --- Comment #2 from Eyal --- Let me know if there's anything that you'd like me to do here! The patch and testing is pretty straight-forward. I think that the new behavior will be more inline with valgrind user expectation. -- You are receiving

[dolphin] [Bug 462443] Crash when renaming a file

2024-02-06 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=462443 Eyal changed: What|Removed |Added Status|NEEDSINFO |RESOLVED Resolution|WAITINGFORINFO

[dolphin] [Bug 462443] Crash when renaming a file

2024-02-06 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=462443 --- Comment #2 from Eyal --- (In reply to Akseli Lahtinen from comment #1) > Have you encountered this bug again on newer version of dolphin? > > Also if you have, can you send a backtrace with debug symbols: > https://commu

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-30 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 Eyal changed: What|Removed |Added Attachment #162607|0 |1 is obsolete

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-30 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #13 from Eyal --- > I think my only question is whether we should "clear" all of data or just > the value fields of the result and opnds[] of the test_data_t? > I guess it doesn't matter whether the rest of the test_d

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-28 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #11 from Eyal --- Comment on attachment 162608 --> https://bugs.kde.org/attachment.cgi?id=162608 This might fix it, too, but I prefer the other one. This one is bigger and the other one works just as well so don't use this one. --

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-28 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #10 from Eyal --- Comment on attachment 162608 --> https://bugs.kde.org/attachment.cgi?id=162608 This might fix it, too, but I prefer the other one. This one is much more convoluted and unnecessary. Just use the other one. --

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-28 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #9 from Eyal --- Created attachment 162619 --> https://bugs.kde.org/attachment.cgi?id=162619=edit A test case to force this error even on amd64. This is just for testing purposes. Don't put this into the repo. -- You are receiv

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-28 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #8 from Eyal --- Drawing up the truth table of those two versions of calculating a_min, I find that only the case when aa is unknown and vaa is 1 make any difference. The output in the version aa & ~vaa return 0 is aa==x and va

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-27 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #7 from Eyal --- Created attachment 162608 --> https://bugs.kde.org/attachment.cgi?id=162608=edit This might fix it, too, but I prefer the other one. We refactor the code so that the computation is done before the vbits are

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-27 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #6 from Eyal --- Created attachment 162607 --> https://bugs.kde.org/attachment.cgi?id=162607=edit This might fix the problem and it's the solution that I prefer. This might fix the problem and it's the solution that I prefer. After

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-27 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 --- Comment #5 from Eyal --- Like I said before: As part of the tests, some vbits get set. However, we need to clear them again before we compute the expectation or else the expectation *itself* will have vbits set in it. The expectation should have

[valgrind] [Bug 417993] vbit-test fail on s390x with Iop_Add32: spurious dependency on uninitialised value

2023-10-27 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=417993 Eyal changed: What|Removed |Added CC||eyals...@gmail.com --- Comment #4 from Eyal

[valgrind] [Bug 476025] Vbit expected test results for Iop_CmpGT64Ux2 are wrong.

2023-10-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=476025 Eyal changed: What|Removed |Added Attachment #162550|0 |1 is obsolete

[valgrind] [Bug 476025] Vbit expected test results for Iop_CmpGT64Ux2 are wrong.

2023-10-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=476025 --- Comment #8 from Eyal --- I completed the new tests. Please try adding ppc platforms and test them. I already added amd64 testing for a few of them. On Wed, Oct 25, 2023, 09:37 Carl Love wrote: > https://bugs.kde.org/show_bug.cgi?id=476

[valgrind] [Bug 476025] Vbit expected test results for Iop_CmpGT64Ux2 are wrong.

2023-10-24 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=476025 --- Comment #6 from Eyal --- Created attachment 162550 --> https://bugs.kde.org/attachment.cgi?id=162550=edit Fix vbits for cmpgtExB where E*B==128 This adds more precise testing to the vbits program. Only amd64 is enabled. Add .ppc32=1 or .pp

[valgrind] [Bug 476025] Vbit expected test results for Iop_CmpGT64Ux2 are wrong.

2023-10-24 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=476025 Eyal changed: What|Removed |Added CC||eyals...@gmail.com --- Comment #5 from Eyal --- I'll

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-23 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #68 from Eyal --- Thanks for doing this! Now this one: https://bugs.kde.org/show_bug.cgi?id=474160 ? Eyal On Tue, Sep 19, 2023 at 11:33 AM Paul Floyd wrote: > https://bugs.kde.org/show_bug.cgi?id=432801 > > Paul Floyd

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-13 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #65 from Eyal --- Yeah, I saw that you have been busy with that! Do you prefer that I go through the process of pushing a branch for CI? I need to get added with some permissions? Eyal On Sun, Sep 10, 2023, 10:02 Paul Floyd wrote

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-07 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #63 from Eyal --- I think that you could have used 'git apply' also. Anyway, I hope that the tests are all passing now and perhaps we can make progress on the patch? Eyal On Wed, Sep 6, 2023, 23:39 Paul Floyd wrote: > ht

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-05 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #61 from Eyal --- (In reply to Paul Floyd from comment #59) > You forgot to update the expecteds (I saw that just the line numbers have > changed), and I had to convert the text .cpp files into symlinks (does Linux > / GCC accep

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-05 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #136381|0 |1 is obsolete

[valgrind] [Bug 474160] If errors-for-leak-kinds is specified, exit-on-first-error should only exit on one of the listed errors.

2023-09-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=474160 --- Comment #1 from Eyal --- Created attachment 161405 --> https://bugs.kde.org/attachment.cgi?id=161405=edit patch with working code and unit test This fixes the issue and adds a unit test. Also here: https://github.com/eyal0/valgrind/pull/5/fi

[valgrind] [Bug 474160] New: If errors-for-leak-kinds is specified, exit-on-first-error should only exit on one of the listed errors.

2023-09-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=474160 Bug ID: 474160 Summary: If errors-for-leak-kinds is specified, exit-on-first-error should only exit on one of the listed errors. Classification: Developer tools Product:

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #139036|0 |1 is obsolete

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #57 from Eyal --- I've identified the problem and I hope to have a solution, soon. It is based on a similar solution for AMD64. Both platforms to not have an xmm shift with lane size of 8. I'll update the patch and also add test cases

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #56 from Eyal --- I was able to get the tests to build and pass on master by running `sudo apt-get install libc6-dbg:i386` and `sudo apt-get remove libc6-dbg`. I guess that this is an x86 and not amd64 problem only? I'm able

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #55 from Eyal --- (In reply to Eyal from comment #54) > (In reply to Mark Wielaard from comment #53) > > (In reply to Eyal from comment #52) > > > Y'all need CI, I can't even build the master branch. :-( > > &

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #54 from Eyal --- (In reply to Mark Wielaard from comment #53) > (In reply to Eyal from comment #52) > > Y'all need CI, I can't even build the master branch. :-( > > We do have CI against various distros and

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #52 from Eyal --- Y'all need CI, I can't even build the master branch. :-( -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2023-09-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #51 from Eyal --- I'll take a look. I gotta say, this patch is super annoying. I've been working on it for 2.5 years now? Instead of github, emails. And no CI. :-( I've resorted to my own personal branch of valgrind with all my fixes

[digikam] [Bug 472879] Grouping: add Group to main Items menu

2023-08-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472879 --- Comment #2 from Eyal --- Two additional advantages: 1. As Maik has pointed out (in another bug), main-menu items are available as toolbar actions (ie. can be added as toolbar buttons). 2. Similarly, main-menu items can be assigned keyboard

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with version

2023-08-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 Eyal changed: What|Removed |Added Summary|Grouping/Versioning: add|Grouping/Versioning: add |grouping

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #12 from Eyal --- I think it should be standalone. Regarding priority it's of course your decision. I can only ask you to consider this: Digikam has versioning. Digikam has grouping. But the two features don't work together. If I batch

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #10 from Eyal --- Bug 338882 - is mostly ignored from 2014, and it is about *automatic* grouping (ie. no user intervention). Bug 318357 - I still don't understand if it covers the Digikam's own versioning. -- You are receiving this mail

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #8 from Eyal --- But I asked if the fix for bug 318357 will handle filename_v1, filename_v2, etc. - you answered it will not. I'm a bit confused. -- You are receiving this mail because: You are watching all bug changes.

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #6 from Eyal --- So why is this a duplicate of the other bug? I still wish to file a bug to have grouping that supports Digikam's versioning scheme. As it is, I can't file this bug because you mark it as a duplicate of another bug, which

[digikam] [Bug 472884] Grouping/Versioning: add Batch Queue Manager option to automatically group new version with original

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472884 --- Comment #2 from Eyal --- Only need to implement grouping function once, and call it from relevant tools. -- You are receiving this mail because: You are watching all bug changes.

[digikam] [Bug 472883] Grouping/Versioning: add Image Editor option to automatically group new version with original

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472883 --- Comment #2 from Eyal --- Only need to implement grouping function once, and call it from relevant tools. -- You are receiving this mail because: You are watching all bug changes.

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #4 from Eyal --- So the fix for bug 318357 will also group these? filename.raw filename_v1.jpg filename_v2.jpg -- You are receiving this mail because: You are watching all bug changes.

[digikam] [Bug 423993] Allow rightclick on thumbnails in preview mode

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=423993 Eyal changed: What|Removed |Added CC||e...@zvi.org --- Comment #7 from Eyal --- I strongly

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 --- Comment #2 from Eyal --- I understand that technically my suggestion is a private case of a more generic / RegEx grouping. However, from a user perspective, grouping by Digikam's standard versioning scheme is less error prone than RegEx, priovides

[digikam] [Bug 472884] New: Grouping/Versioning: add Batch Queue Manager option to automatically group new version with original

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472884 Bug ID: 472884 Summary: Grouping/Versioning: add Batch Queue Manager option to automatically group new version with original Classification: Applications Product: digikam Version:

[digikam] [Bug 472883] Grouping/Versioning: add Image Editor option to automatically group new version with original

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472883 Eyal changed: What|Removed |Added Platform|Other |Archlinux -- You are receiving this mail because: You

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 Eyal changed: What|Removed |Added Platform|Other |Archlinux -- You are receiving this mail because: You

[digikam] [Bug 472883] New: Grouping/Versioning: add Image Editor option to automatically group new version with original

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472883 Bug ID: 472883 Summary: Grouping/Versioning: add Image Editor option to automatically group new version with original Classification: Applications Product: digikam Version: unspecified

[digikam] [Bug 472881] Grouping/Versioning: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 Eyal changed: What|Removed |Added Summary|Grouping: add grouping by |Grouping/Versioning: add |filename

[digikam] [Bug 472881] New: Grouping: add grouping by filename with versioning

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472881 Bug ID: 472881 Summary: Grouping: add grouping by filename with versioning Classification: Applications Product: digikam Version: unspecified Platform: Other OS: Linux

[digikam] [Bug 472879] New: Grouping: add Group to main Items menu

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472879 Bug ID: 472879 Summary: Grouping: add Group to main Items menu Classification: Applications Product: digikam Version: unspecified Platform: Archlinux OS: All Status:

[digikam] [Bug 472878] New: Grouping: add toolbar buttons / actions

2023-08-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=472878 Bug ID: 472878 Summary: Grouping: add toolbar buttons / actions Classification: Applications Product: digikam Version: unspecified Platform: Archlinux OS: All Status:

[amarok] [Bug 399037] Repeated "QGLContext::makeCurrent(): Cannot make invalid context current” and hang

2022-11-30 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=399037 Eyal Rozenberg changed: What|Removed |Added Status|NEEDSINFO |REPORTED Resolution|WAITINGFORINFO

[dolphin] [Bug 462443] New: Crash when renaming a file

2022-11-30 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=462443 Bug ID: 462443 Summary: Crash when renaming a file Classification: Applications Product: dolphin Version: 22.08.3 Platform: Archlinux OS: Linux Status: REPORTED

[kile] [Bug 392247] Structure tree construction does not ignore sectioning within comment environment

2022-11-18 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=392247 Eyal Rozenberg changed: What|Removed |Added Resolution|WAITINGFORINFO |--- Status|NEEDSINFO

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2022-02-10 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #47 from Eyal --- Any progress here? I see the valgrind is getting new commits all the time. -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-07-24 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #45 from Eyal --- Sure, let me know if I can be of assistance. This was an actual issue for me with false positives in boost in newer clang. Boost is quite popular so I imagine that as more people upgrade to newer clang, maybe more

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-06-06 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #41 from Eyal --- Got it done! Thank you for testing out the code. Try again? -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-06-06 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #138226|0 |1 is obsolete

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-06-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #37 from Eyal --- Any further comments? This is fixing a real false positive that users of clang will experience with reasonable optimization levels. -- You are receiving this mail because: You are watching all bug changes.

[konsole] [Bug 370397] Support coloring/tinting of tabs

2021-05-11 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=370397 --- Comment #4 from Eyal Rozenberg --- In those screenshot, I see a colored line under the tab. That's not sufficient, it should be possible to color the whole tab. The line is not noticeable enough. -- You are receiving this mail because: You

[konsole] [Bug 370397] Support coloring/tinting of tabs

2021-05-11 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=370397 --- Comment #2 from Eyal Rozenberg --- (In reply to Podagric from comment #1) > it is now possible to do so. > could you close this bug? I'm not using the latest KDE right now, so I can't verify this being the case. But I won't object if s

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-05-07 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #36 from Eyal --- Oh, I see it now. I must have forgotten to recompile. I've sent the new patch. -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-05-07 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #137990|0 |1 is obsolete

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-05-07 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #34 from Eyal --- Huh. I don't have any errors on my computer with it. Can you send an example error from the test? Maybe the location of the conditional jump depends on the platform...? -- You are receiving this mail because: You

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-05-07 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #32 from Eyal --- How does the latest code look? Are all the warnings gone now? Passes tests on FreeBSD? -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-04-28 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #136599|0 |1 is obsolete

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-04-21 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #29 from Eyal --- Okay. I don't think that I'm using any special features of c nor c++... How would I know? Is there some test that I can run? I didn't find any errors when I compiled with -Wall -Werror so I didn't see that string being

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-04-20 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #27 from Eyal --- Maybe that's with the c code. I wrote a version in c++ which is uses templates to make the test code much shorter. Let's focus on just one of them so that I don't have to duplicate too much work. Do we want c

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-11 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #25 from Eyal --- Created attachment 136599 --> https://bugs.kde.org/attachment.cgi?id=136599=edit patch that supports pcmpgtX for 64/32/16/8 bit testing all lanes and using templates This one uses c++ in the test only. It is a m

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-10 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #24 from Eyal --- Created attachment 136575 --> https://bugs.kde.org/attachment.cgi?id=136575=edit patch that supports pcmpgtX for 64/32/16/8 bit This one is more complex but it supports all the different pcmpgt* instructions in

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-10 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #23 from Eyal --- Generalizing the patch to handle different bit widths would be easy. In fact, that's why I wrote it using a switch statement with just one case. To generalize it you just need to add cases into the switch statement. Pretty

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-09 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #21 from Eyal --- Will anyone consider the patch? There might be a lot of false positives out there now that clang is doing this optimization and it would be nice to catch them before many users waste a lot of time on this. -- You

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #20 from Eyal --- Created attachment 136381 --> https://bugs.kde.org/attachment.cgi?id=136381=edit patch with tests -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #19 from Eyal --- Comment on attachment 136346 --> https://bugs.kde.org/attachment.cgi?id=136346 patch for expensive greater than comparisons Obsolete now. -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #136346|0 |1 is obsolete

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-04 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #18 from Eyal --- >Interesting analysis, and a plausible patch; thank you for that. This seems >like a new trick from LLVM. Thanks. Yes >* where's the addition instruction that merges the lanes together? I don't > see that

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added Attachment #136346|path for expensive greater |patch for expensive greater description|than

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-03 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #16 from Eyal --- Created attachment 136346 --> https://bugs.kde.org/attachment.cgi?id=136346=edit path for expensive greater than comparisons -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-03-01 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #14 from Eyal --- A possible solution in Valgrind: https://github.com/eyal0/valgrind/commit/899ea491e358013579f87e443beff0a30c69e348 This improves Valgrind's check for definedness when doing 32x4 SIMD signed greater than. It solves

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-27 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #13 from Eyal --- https://bugs.llvm.org/show_bug.cgi?id=49372 I hope that LLVM can help us out here! I think that it would be nice. -- You are receiving this mail because: You are watching all bug changes.

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-26 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #12 from Eyal --- Okay, now I'm back to thinking that it's a valgrind issue. But it's nothing that valgrind can fix. Here's the problematic asm code: movd %edx,%xmm2 (1) punpcklbw %xmm2,%xmm2 (2

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-26 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #11 from Eyal --- I'm back to not being sure if this is an llvm issue. There are a few things at play here. One is sigaction(), which can foul up the the contents of xmm registers, especially xmm3, which is the only register that isn't

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #9 from Eyal --- Oops, I spoke to soon. It's a bug in clang. Here's code that you can try: #include #include #include #include void foo() { // Put the garbage number 123 into eax. // It's caller-saved so no problem

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #8 from Eyal --- Even vgdb isn't helping me. Here's the code that I'm using: #include #include #include #include int main() { struct sigaction act; if (sigaction(SIGTERM, 0, ) == 1) { return 12; } if (sigaction(SIGTERM, 0

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #7 from Eyal --- It seems that the issue is the loop. When the length of pattern is more than 8, the code runs a routine that is able to sum 8 chars at a time. It uses xmm for this. It only enters that code if the number of bytes to sum

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-25 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 --- Comment #6 from Eyal --- Here's one that also fails but with a different error: ==12859== Syscall param exit_group(status) contains uninitialised byte(s) ==12859==at 0x492F9D6: _Exit (_exit.c:31) ==12859==by 0x48A2E89: __run_exit_handlers

[valgrind] [Bug 432801] Valgrind 3.16.1 reports a jump based on uninitialized memory somehow related to clang and signals

2021-02-24 Thread Eyal
https://bugs.kde.org/show_bug.cgi?id=432801 Eyal changed: What|Removed |Added CC||eyals...@gmail.com --- Comment #5 from Eyal --- I have

[amarok] [Bug 409837] CMakeLists.txt doesn't check for presence of Embedded MariaDB

2021-02-13 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409837 Eyal Rozenberg changed: What|Removed |Added Resolution|WAITINGFORINFO |WORKSFORME Status|NEEDSINFO

[amarok] [Bug 409837] CMakeLists.txt doesn't check for presence of Embedded MariaDB

2021-02-13 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409837 --- Comment #4 from Eyal Rozenberg --- (In reply to Heiko Becker from comment #3) > As I wrote 2.9.71 (aka 2.10 Alpha1, you can get it from > https://download.kde.org/unstable/amarok/2.9.71/). git master works too and > isn't that diffe

[amarok] [Bug 409837] CMakeLists.txt doesn't check for presence of Embedded MariaDB

2021-02-13 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409837 --- Comment #2 from Eyal Rozenberg --- (In reply to Heiko Becker from comment #1) Which exact version should I try and build? -- You are receiving this mail because: You are watching all bug changes.

[extra-cmake-modules] [Bug 409966] Check if plugin install path is where the system expects them

2019-08-06 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 --- Comment #14 from Eyal Rozenberg --- Ah, I see what you mean. Ok then. That is, I'm not pleased but at least I understand the rationale of your decisions. -- You are receiving this mail because: You are watching all bug changes.

[extra-cmake-modules] [Bug 409966] Check if plugin install path is where the system expects them

2019-08-06 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 --- Comment #12 from Eyal Rozenberg --- Excuse me, but - are you serious? This is the _Amarok_ bug tracking system. You can't close bugs about the current release and tell people not to use it. If that's how you feel - terminate your account or stop

[extra-cmake-modules] [Bug 409966] Check if plugin install path is where the system expects them

2019-08-06 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 --- Comment #10 from Eyal Rozenberg --- (In reply to Christophe Giboudeaux from comment #9) > Doesn't change anything. There's no officially supported Qt5 based amarok The single current official release of Amarok is 2.9.0. That's the

[extra-cmake-modules] [Bug 409966] Check if plugin install path is where the system expects them

2019-08-06 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 Eyal Rozenberg changed: What|Removed |Added Resolution|NOT A BUG |--- Status|RESOLVED

[extra-cmake-modules] [Bug 409966] Check if plugin install path is where the system expects them

2019-08-05 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 --- Comment #6 from Eyal Rozenberg --- (In reply to Christophe Giboudeaux from comment #4) > why ecm ? isn't amarok 2.9 the KDE4 version? Can you explain the latest exchange of comments? I'm not quite following. Amarok 2.9 is the current rele

[amarok] [Bug 409966] Amarok won't run, complains about missing plugins

2019-08-05 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409966 Eyal Rozenberg changed: What|Removed |Added Status|RESOLVED|REOPENED Ever confirmed|0

[amarok] [Bug 409966] New: Amarok won't run, complains about missing plugins

2019-07-18 Thread Eyal Rozenberg
Severity: major Priority: NOR Component: general Assignee: amarok-bugs-d...@kde.org Reporter: eyal...@technion.ac.il Target Milestone: kf5 I've downloaded and built Amarok 2.9.0 on Devuan Beowulf (= Debian Buster without systemd) - after installing a whole

[amarok] [Bug 409836] Build failure: Undefined reference to setMountPointManager(MountPointManager*)

2019-07-18 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409836 --- Comment #4 from Eyal Rozenberg --- (In reply to Myriam Schweingruber from comment #3) > I guessed as much. It's still a build failure, albeit of the testing code. Aren't you supposed to change the component or mark it somehow instead of test

[amarok] [Bug 409836] Build failure: Undefined reference to setMountPointManager(MountPointManager*)

2019-07-16 Thread Eyal Rozenberg
https://bugs.kde.org/show_bug.cgi?id=409836 Eyal Rozenberg changed: What|Removed |Added Status|NEEDSINFO |REPORTED Resolution|WAITINGFORINFO

[amarok] [Bug 409837] New: CMakeLists.txt doesn't check for presence of Embedded MariaDB

2019-07-15 Thread Eyal Rozenberg
: REPORTED Severity: normal Priority: NOR Component: general Assignee: amarok-bugs-d...@kde.org Reporter: eyal...@technion.ac.il Target Milestone: kf5 SUMMARY I'm building v2.9.0 on Debian buster (well, actually, Devuan Beowulf, which is the same

  1   2   >