Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update GCutil for fix performance problem #1320

Merged
merged 2 commits into from
Feb 14, 2024
Merged

Conversation

ksh8281
Copy link
Contributor

@ksh8281 ksh8281 commented Feb 6, 2024

There was a problem running bubble test in web-tooling-benchmark too many gc were occured because GC_get_bytes_since_gc() cannot track GC_free event and fixes build error with mem_stats profiler(valgrind)

@ksh8281 ksh8281 changed the title Update GCutil for fix performance problem [wip]Update GCutil for fix performance problem Feb 6, 2024
@ksh8281 ksh8281 force-pushed the work66 branch 4 times, most recently from 8fdf3bb to b4f7037 Compare February 8, 2024 01:35
There was a problem running bubble test in web-tooling-benchmark
too many gc were occured because GC_get_bytes_since_gc() cannot track GC_free event
so, I am start to use GC_REALLOC on 64-32bit
and fixes build error with mem_stats profiler(valgrind)

Signed-off-by: Seonghyun Kim <sh8281.kim@samsung.com>
Signed-off-by: Seonghyun Kim <sh8281.kim@samsung.com>
@ksh8281 ksh8281 changed the title [wip]Update GCutil for fix performance problem Update GCutil for fix performance problem Feb 14, 2024
Copy link
Contributor

@clover2123 clover2123 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@clover2123 clover2123 merged commit 6555853 into Samsung:master Feb 14, 2024
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants