https://bugs.kde.org/show_bug.cgi?id=398445
Ivo Raisr changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comment #6 from Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=396290
--- Comment #20 from Ivo Raisr ---
Thank you for your work. This is going to be a useful Valgrind tool.
I like the documentation for the latest patch. Reasoning explained well on
examples. You could mention that the default behaviour (fail unseen
https://bugs.kde.org/show_bug.cgi?id=396290
--- Comment #14 from Ivo Raisr ---
The current exp-allocfail crashes badly on my Ubuntu 18.04 box.
When running './vg-in-place --tool=exp-allocfail /bin/date', it crashes at
af_main.c:119.
That's because i is equal to an equivalent of '-1' (4294967295
https://bugs.kde.org/show_bug.cgi?id=396290
--- Comment #13 from Ivo Raisr ---
(In reply to roger.light from comment #8)
> Created attachment 114229 [details]
> Updated patch with fixes and text callstacks.
Looks quite good. There are just few nits:
- Please remove trailing whitespa
https://bugs.kde.org/show_bug.cgi?id=396290
--- Comment #10 from Ivo Raisr ---
(In reply to Philippe Waroquiers from comment #9)
> I am wondering also how much difficult it would be to somewhat more
> generalise the tool.
>
> For example, we might want to make similar tests to c
https://bugs.kde.org/show_bug.cgi?id=385409
Ivo Raisr changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comment #19 from Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=396290
--- Comment #7 from Ivo Raisr ---
Alright, I think it would be preferable to have --num-callers used instead
of --depth. Users are already familiar with --num-callers.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=384337
Ivo Raisr changed:
What|Removed |Added
Status|CONFIRMED |RESOLVED
Resolution
https://bugs.kde.org/show_bug.cgi?id=396290
Ivo Raisr changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCONFIRMED
https://bugs.kde.org/show_bug.cgi?id=381553
--- Comment #75 from Ivo Raisr ---
And thank you, Julian, for your guidance.
Unfortunately I won't be able to finish the VEX enhancement/rework residing in
(I think) vex-jit-hacks-2 branch. Perhaps some brave soul will be able to
continue...
--
You
https://bugs.kde.org/show_bug.cgi?id=395682
Ivo Raisr changed:
What|Removed |Added
CC||ori...@gmx.fr
--- Comment #14 from Ivo Raisr
https://bugs.kde.org/show_bug.cgi?id=384727
Ivo Raisr changed:
What|Removed |Added
Resolution|--- |DUPLICATE
Status|ASSIGNED
https://bugs.kde.org/show_bug.cgi?id=384727
Ivo Raisr changed:
What|Removed |Added
Ever confirmed|0 |1
Assignee|jsew...@acm.org
https://bugs.kde.org/show_bug.cgi?id=390471
Ivo Raisr changed:
What|Removed |Added
Resolution|--- |FIXED
Status|ASSIGNED
https://bugs.kde.org/show_bug.cgi?id=390471
Ivo Raisr changed:
What|Removed |Added
Attachment #110691|0 |1
is obsolete
https://bugs.kde.org/show_bug.cgi?id=390471
Ivo Raisr changed:
What|Removed |Added
Status|UNCONFIRMED |ASSIGNED
Ever confirmed|0
https://bugs.kde.org/show_bug.cgi?id=197915
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comme
https://bugs.kde.org/show_bug.cgi?id=338252
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comme
https://bugs.kde.org/show_bug.cgi?id=390524
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
https://bugs.kde.org/show_bug.cgi?id=390524
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivosh.net
-
https://bugs.kde.org/show_bug.cgi?id=388862
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Reso
https://bugs.kde.org/show_bug.cgi?id=388862
--- Comment #10 from Ivo Raisr <iv...@ivosh.net> ---
:-)
There was also a problem with the actual replacement definition.
For example wcsnlen had the following:
#if defined(VGO_linux)
STRNLEN(VG_Z_LIBC_SONAME, wcsnlen)
STRNLEN(VG_Z_LIBC_
https://bugs.kde.org/show_bug.cgi?id=388862
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #42|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=388862
--- Comment #7 from Ivo Raisr <iv...@ivosh.net> ---
Oh, alright, let's make it Int also for wcslen() so the whole file is
consistent.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=388862
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #111086|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=388862
--- Comment #4 from Ivo Raisr <iv...@ivosh.net> ---
Thank you for the patch.
I had to make some fixes to make it actually work and do not introduce
regressions. Please review the attachment.
--
You are receiving this mail because:
You are watchi
https://bugs.kde.org/show_bug.cgi?id=388862
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivo
https://bugs.kde.org/show_bug.cgi?id=388862
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivo
https://bugs.kde.org/show_bug.cgi?id=390723
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |FIXED
Status|AS
https://bugs.kde.org/show_bug.cgi?id=390723
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCON
https://bugs.kde.org/show_bug.cgi?id=390723
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivosh.net
-
https://bugs.kde.org/show_bug.cgi?id=390723
Bug ID: 390723
Summary: make xtree dump files world wide readable, similar to
log files
Product: valgrind
Version: 3.14 SVN
Platform: Compiled Sources
OS: Linux
https://bugs.kde.org/show_bug.cgi?id=389065
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |FIXED
Status|AS
https://bugs.kde.org/show_bug.cgi?id=384633
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|CONFIRMED |RESOLVED
Reso
https://bugs.kde.org/show_bug.cgi?id=384633
--- Comment #7 from Ivo Raisr <iv...@ivosh.net> ---
Hi Julian, patch with Solaris implementation is attached.
The patch has been refreshed to apply cleanly to the latest git HEAD and a NEWS
entry added. It is ready to be pushed - let me know if I
https://bugs.kde.org/show_bug.cgi?id=384633
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #109872|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=389065
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|CONFIRMED |ASSIGNED
--- Comment #4 fr
https://bugs.kde.org/show_bug.cgi?id=389065
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384633
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|UNCONFIRMED |CONFIRMED
Ever confi
https://bugs.kde.org/show_bug.cgi?id=389065
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivo
https://bugs.kde.org/show_bug.cgi?id=388895
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=385408
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivo
https://bugs.kde.org/show_bug.cgi?id=388493
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=381819
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||x...@fictionpress.com
--- C
https://bugs.kde.org/show_bug.cgi?id=328357
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |DUPLICATE
https://bugs.kde.org/show_bug.cgi?id=381819
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||sgtcapsl...@lavab
https://bugs.kde.org/show_bug.cgi?id=322586
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |DUPLICATE
https://bugs.kde.org/show_bug.cgi?id=381819
--- Comment #16 from Ivo Raisr <iv...@ivosh.net> ---
Created attachment 109550
--> https://bugs.kde.org/attachment.cgi?id=109550=edit
support for TBM bextr
Patch originally attached to bug https://bugs.kde.org/show_bug.cgi?id=322586
by &q
https://bugs.kde.org/show_bug.cgi?id=381819
--- Comment #15 from Ivo Raisr <iv...@ivosh.net> ---
(In reply to Bernd Buschinski from comment #14)
> any chance we can push/fix/update this to official valgrind? I would happy
> to test new patches :)
As discussed previously, the pa
https://bugs.kde.org/show_bug.cgi?id=387766
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=208052
--- Comment #9 from Ivo Raisr <iv...@ivosh.net> ---
Fixed in commit a5af4146e383dd442130905901b046e2cb4b0ed4,
https://sourceware.org/git/gitweb.cgi?p=valgrind.git;h=a5af4146e383dd442130905901b046e2cb4b0ed4
NEWS updated in
https://bugs.kde.org/show_bug.cgi?id=208052
--- Comment #7 from Ivo Raisr <iv...@ivosh.net> ---
Tom, would you mind integrating the fix into Valgrind source code base at this
point? Alternatively, I could do it myself - just let me know.
--
You are receiving this mail because:
You are wa
https://bugs.kde.org/show_bug.cgi?id=208052
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCON
https://bugs.kde.org/show_bug.cgi?id=385939
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |FIXED
Status|AS
https://bugs.kde.org/show_bug.cgi?id=386318
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Assignee|jsew...@acm.org |iv...@ivosh.net
-
https://bugs.kde.org/show_bug.cgi?id=386318
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|UNCONFIRMED |RESOLVED
Reso
https://bugs.kde.org/show_bug.cgi?id=376257
--- Comment #5 from Ivo Raisr <iv...@ivosh.net> ---
Nice work, Philippe!
I have just one question. In helgrind/hg_main.c, the last hunk,
there is a message containing a weird character: "re-àsetting it to 0".
Is this intended?
--
Y
https://bugs.kde.org/show_bug.cgi?id=386318
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=385939
--- Comment #2 from Ivo Raisr <iv...@ivosh.net> ---
Thank you for the patch and kudos for documenting the new option in the manual.
>From the high level perspective, I have the following comments and questions:
* It is not clear wha
https://bugs.kde.org/show_bug.cgi?id=385939
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Component|memcheck|general
Summary|
https://bugs.kde.org/show_bug.cgi?id=385939
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivo
https://bugs.kde.org/show_bug.cgi?id=384676
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |WONTFIX
https://bugs.kde.org/show_bug.cgi?id=385843
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Ever confirmed|0 |1
Status|UNCON
https://bugs.kde.org/show_bug.cgi?id=385843
--- Comment #9 from Ivo Raisr <iv...@ivosh.net> ---
(In reply to Sindre Aamås from comment #6)
> I have not looked at the stats, but the output is as follows.
As suspected, there is a code bloat in the generated code because of the
additional
https://bugs.kde.org/show_bug.cgi?id=385843
--- Comment #5 from Ivo Raisr <iv...@ivosh.net> ---
Your fix touches register allocation. It is crucial that both ARMInstr_Call()
and getRRegUniverse_ARM() are kept in sync (hinted in getRRegUniverse_ARM as
well)
and that register all
https://bugs.kde.org/show_bug.cgi?id=385843
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #21 from Ivo Raisr <iv...@ivosh.net> ---
Thank you all for your responses! They were really helpful.
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|ASSIGNED|RESOLVED
Reso
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #19 from Ivo Raisr <iv...@ivosh.net> ---
(In reply to Julian Seward from comment #17)
> I expect arm(32) would also get a quite big improvement, because
> it's also pretty low on registers. [This is not a request to measure,
&
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #16 from Ivo Raisr <iv...@ivosh.net> ---
I've fixed the problem with duplicate hregIsVirtual() in host_arch_defs.c
files.
Here are the performance numbers for amd64 and x86:
Running inner Memcheck on perf/bz2 (compiled with -O
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #108135|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #108134|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #6 from Ivo Raisr <iv...@ivosh.net> ---
I've tested on amd64, ppc8le and arm64 architectures.
In all cases, the produced code was more compact and overall performance
better, when running inner Memcheck on perf/bz2.
Numbers are
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #107964|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #107963|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
https://bugs.kde.org/show_bug.cgi?id=384337
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #3 from Ivo Raisr <iv...@ivosh.net> ---
The only remaining thing is to refactor isMove callbacks in the remaining
architectures so that the information is available directly in HRegUsage.
--
You are receiving this mail becaus
https://bugs.kde.org/show_bug.cgi?id=384987
--- Comment #2 from Ivo Raisr <iv...@ivosh.net> ---
Created attachment 107964
--> https://bugs.kde.org/attachment.cgi?id=107964=edit
register allocation
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Status|CONFIRMED |ASSIGNED
--
You are rec
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384987
Bug ID: 384987
Summary: VEX register allocator: allocate caller-save registers
for short lived vregs
Product: valgrind
Version: 3.14 SVN
Platform: Compiled Sources
https://bugs.kde.org/show_bug.cgi?id=384987
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Ever confirmed|0 |1
Assigne
https://bugs.kde.org/show_bug.cgi?id=384337
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Attachment #107674|0 |1
is ob
https://bugs.kde.org/show_bug.cgi?id=384732
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384729
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384584
--- Comment #6 from Ivo Raisr <iv...@ivosh.net> ---
At the end only %r11 was removed from the list in getRegUsage_AMD64Instr()
because it's used overall as a temporary register for destination address.
%rax, %rcx, %rdx could in theory b
https://bugs.kde.org/show_bug.cgi?id=384584
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Resolution|--- |FIXED
Status|AS
https://bugs.kde.org/show_bug.cgi?id=384676
--- Comment #2 from Ivo Raisr <iv...@ivosh.net> ---
Unfortunately performance measurements do not confirm this as a good idea.
Measuring Memcheck on perf/bz2, instruction count:
v3 baseline: 45,110 M total; 168 M register allocator
v3 patched:
https://bugs.kde.org/show_bug.cgi?id=384584
--- Comment #4 from Ivo Raisr <iv...@ivosh.net> ---
The part with %rax, %rcx, %rdx, %r11 removal from getRegUsage_AMD64Instr() will
be removed from the patch.
The reason is explained in greater detail in:
Bug 384676 - VEX AMD64 backend should lis
https://bugs.kde.org/show_bug.cgi?id=384584
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
https://bugs.kde.org/show_bug.cgi?id=384676
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
https://bugs.kde.org/show_bug.cgi?id=384676
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384676
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
Ever confirmed|0 |1
Assigne
https://bugs.kde.org/show_bug.cgi?id=384676
Bug ID: 384676
Summary: VEX AMD64 backend should list more real registers as
available for the register allocator
Product: valgrind
Version: 3.14 SVN
Platform: Compiled Sources
https://bugs.kde.org/show_bug.cgi?id=384633
--- Comment #3 from Ivo Raisr <iv...@ivosh.net> ---
And also please add the line explanation somewhere. User manual, verbose
output...
--
You are receiving this mail because:
You are watching all bug changes.
https://bugs.kde.org/show_bug.cgi?id=384633
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384584
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
CC||iv...@ivosh.net
--- Comm
https://bugs.kde.org/show_bug.cgi?id=384584
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
https://bugs.kde.org/show_bug.cgi?id=384337
Ivo Raisr <iv...@ivosh.net> changed:
What|Removed |Added
See Also||https://bugs.kde.org/
1 - 100 of 599 matches
Mail list logo
