Update build system to support 64-bit ASan DLLs

RESOLVED FIXED in Firefox 52

Status

defect
RESOLVED FIXED
3 years ago
a year ago

People

(Reporter: dmajor, Assigned: dmajor)

Tracking

(Blocks 1 bug)

unspecified
mozilla52
Dependency tree / graph

Firefox Tracking Flags

(firefox52 fixed)

Details

Attachments

(1 attachment)

Assignee

Description

3 years ago
Currently we have clang_rt.asan_dynamic-i386.dll hard-coded in a couple of places.
Assignee

Comment 1

3 years ago
Posted patch patchSplinter Review
Assignee: nobody → dmajor
Attachment #8805168 - Flags: review?(ted)
Comment on attachment 8805168 [details] [diff] [review]
patch

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

::: build/autoconf/sanitize.m4
@@ +14,5 @@
>  if test -n "$MOZ_ASAN"; then
>      MOZ_LLVM_HACKS=1
>      if test -n "$CLANG_CL"; then
> +        # Look for the ASan runtime binary
> +        if test -n "$HAVE_64BIT_BUILD"; then

Slight preference for `test "$CPU_ARCH" = "x86_64"` here.
Attachment #8805168 - Flags: review?(ted) → review+

Comment 3

3 years ago
Pushed by dmajor@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/c6587fbc2dbe
Update build system to support 64-bit ASan DLLs. r=ted

Comment 4

3 years ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/c6587fbc2dbe
Status: NEW → RESOLVED
Last Resolved: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla52

Updated

a year ago
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.