Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 74483928 authored by Pirama Arumuga Nainar's avatar Pirama Arumuga Nainar
Browse files

[cc/sanitize] Re-enable new pass manager for fuzzer targets 

Bug: http://b/133876586

This was turned off but the upstream change fixing this has been long
part of the Android toolchain.

Test: Output of
  m aidl_parser_fuzzer; readelf aidl_parser_fuzzer | grep -c sancov
hasn't changed.

Change-Id: Id7edd7ac5791b0e031fd4af46381acdbb1b80d47
parent 93c3cb69

cc/sanitize.go

+0 −4
Original line number Diff line number Diff line
@@ -35,7 +35,6 @@ var ( 


	asanCflags = []string{

		"-fno-omit-frame-pointer",

		"-fno-experimental-new-pass-manager",

	}

	asanLdflags = []string{"-Wl,-u,__asan_preinit"}
@@ -666,9 +665,6 @@ func (sanitize *sanitize) flags(ctx ModuleContext, flags Flags) Flags { 
		flags.Local.LdFlags = append(flags.Local.LdFlags, "-fno-sanitize-coverage=stack-depth")

		flags.Local.CFlags = append(flags.Local.CFlags, "-fno-sanitize-coverage=stack-depth")



		// TODO(b/133876586): Experimental PM breaks sanitizer coverage.

		flags.Local.CFlags = append(flags.Local.CFlags, "-fno-experimental-new-pass-manager")



		// Disable fortify for fuzzing builds. Generally, we'll be building with

		// UBSan or ASan here and the fortify checks pollute the stack traces.

		flags.Local.CFlags = append(flags.Local.CFlags, "-U_FORTIFY_SOURCE")