Closed Bug 1625268 Opened 4 years ago Closed 4 years ago

Experimental new pass manager breaks libFuzzer

Categories

(Firefox Build System :: Toolchains, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(firefox76 fixed)

RESOLVED FIXED
mozilla76
Tracking Status
firefox76 --- fixed

People

(Reporter: decoder, Assigned: decoder)

Details

Attachments

(1 file)

It looks like -fexperimental-new-pass-manager has the potential to break libFuzzer instrumentation: I was just debugging an issue on oss-fuzz where the resulting build claimed that no instrumentation was detected. I traced it back to the new pass manager and indeed building with --disable-new-pass-manager solved the problem there.

At the same time, our own builds seem to still work for some reason (oss-fuzz uses our toolchain so I really don't know for sure where the difference is). But I suggest that for now, we disable the new pass manager in fuzzing builds.

Could you point me to a broken oss-fuzz build and a successful Mozilla one? I would be interested in examining them.

(In reply to :dmajor from comment #1)

Could you point me to a broken oss-fuzz build and a successful Mozilla one? I would be interested in examining them.

Nevermind, I just fetched another TC build and it also appears to be broken... My local build is ok, but that uses a newer Clang.

Assignee: nobody → choller
Status: NEW → ASSIGNED
Pushed by choller@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c695eb5a7bc4
Disable new pass manager for fuzzing when using older Clang. r=dmajor
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla76
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: