This repository was archived by the owner on Mar 29, 2024. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 14
Valgring reports problem after upgrading to v8 6.3 #47
Labels
Milestone
Comments
It seems to be https://bugs.chromium.org/p/v8/issues/detail?id=6566 bug may be related to this issue. |
pinepain
added a commit
that referenced
this issue
Sep 3, 2017
pinepain
added a commit
that referenced
this issue
Sep 3, 2017
New error come up after suppressing valgrind reports - https://travis-ci.org/pinepain/php-v8/jobs/271439249
|
Reproducing locally using
|
I decided to move away from what travis PHP image offers and use own docker images so that it would be less pita to debug what happens on CI. Valgrind reports specified above as well as some other reports which appears in docker are suppressed for now and could be revisited later when I have time or someone volunteer to do that. Ticket number for that is #49. |
Merged
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
After updating some tests become red and reports leaks - https://travis-ci.org/pinepain/php-v8/jobs/271257124.
I'm not sure was there any changes in v8 internals that affects userland code and thus need some adjustmets or it's a v8 internals issue or it just false-positive. For now I filled a report to v8-users mail lists - https://groups.google.com/forum/#!topic/v8-users/qu1HrjbgRsg.
While we can workaround this issue by simply suppress this issues with valgringrc, it might be a notice of more deeper problem.
The text was updated successfully, but these errors were encountered: