Closed Bug 749768 Opened 12 years ago Closed 12 years ago

Disable --valgrind flag for jit_test.py when running "make check" with AddressSanitizer

Categories

(Core :: JavaScript Engine, defect)

All
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla15

People

(Reporter: decoder, Assigned: decoder)

References

Details

(Whiteboard: [sg:want][asan-build-blocker][asan-test-blocker])

Attachments

(1 file, 1 obsolete file)

In js/src/Makefile.in the jit_test.py script is called with "--valgrind" which causes some tests to timeout because running an ASan binary under valgrind doesn't work.

I'm writing a patch that only passes --valgrind in the Makefile when MOZ_ASAN is not set.
Attached patch Patch (obsolete) — Splinter Review
Assignee: general → choller
Status: NEW → ASSIGNED
Attachment #619528 - Flags: review?(khuey)
Comment on attachment 619528 [details] [diff] [review]
Patch

Review of attachment 619528 [details] [diff] [review]:
-----------------------------------------------------------------

::: js/src/Makefile.in
@@ +605,5 @@
>  ifneq ($(OS_ARCH),WINNT) # FIXME: this should be made work on Windows too.
>  #check:: check-malloc-function-usage FIXME: disable on JM until closer to merge time.
>  endif
>  
> +ifneq ($(MOZ_ASAN),1)

ifndef MOZ_ASAN
Attachment #619528 - Flags: review?(khuey) → review+
Attached patch Updated patchSplinter Review
Updated patch, addresses review comment. Keeping r+ from previous patch version.
Attachment #619528 - Attachment is obsolete: true
Attachment #619535 - Flags: review+
Attachment #619535 - Flags: checkin?(gary)
Marking also as a build blocker because this causes "make check" to run for the maximum allowed time per test using valgrind, rendering the builds very slow.
Whiteboard: [sg:want][asan-build-blocker][asan-test-blocker]
Target Milestone: --- → mozilla15
https://hg.mozilla.org/mozilla-central/rev/1ed79aeea82f
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: