Merge "Refactor sanitized library on-disk layout - Soong."

am: 94e1747051

Change-Id: I823ad49a1f4ed6b0fbb683ffe186ca100700652d
This commit is contained in:
Vishwath Mohan 2017-03-31 16:41:26 +00:00 committed by android-build-merger
commit d6f6ad8073
6 changed files with 37 additions and 11 deletions

View file

@ -89,6 +89,7 @@ type ModuleContext interface {
AddMissingDependencies(deps []string)
InstallInData() bool
InstallInSanitizerDir() bool
RequiredModuleNames() []string
}
@ -103,6 +104,7 @@ type Module interface {
Enabled() bool
Target() Target
InstallInData() bool
InstallInSanitizerDir() bool
SkipInstall()
}
@ -399,6 +401,10 @@ func (p *ModuleBase) InstallInData() bool {
return false
}
func (p *ModuleBase) InstallInSanitizerDir() bool {
return false
}
func (a *ModuleBase) generateModuleTarget(ctx blueprint.ModuleContext) {
allInstalledFiles := Paths{}
allCheckbuildFiles := Paths{}
@ -629,6 +635,10 @@ func (a *androidModuleContext) InstallInData() bool {
return a.module.InstallInData()
}
func (a *androidModuleContext) InstallInSanitizerDir() bool {
return a.module.InstallInSanitizerDir()
}
func (a *androidModuleContext) InstallFileName(installPath OutputPath, name string, srcPath Path,
deps ...Path) OutputPath {

View file

@ -652,7 +652,10 @@ func PathForModuleInstall(ctx ModuleContext, paths ...string) OutputPath {
if ctx.Proprietary() {
partition = ctx.DeviceConfig().VendorPath()
}
if ctx.InstallInData() {
if ctx.InstallInSanitizerDir() {
partition = "data/asan/" + partition
} else if ctx.InstallInData() {
partition = "data"
}
outPaths = []string{"target", "product", ctx.AConfig().DeviceName(), partition}

View file

@ -212,6 +212,7 @@ type installer interface {
installerProps() []interface{}
install(ctx ModuleContext, path android.Path)
inData() bool
inSanitizerDir() bool
hostToolPath() android.OptionalPath
}
@ -993,10 +994,17 @@ func (c *Module) InstallInData() bool {
if c.installer == nil {
return false
}
if c.sanitize != nil && c.sanitize.inData() {
return c.installer.inData()
}
func (c *Module) InstallInSanitizerDir() bool {
if c.installer == nil {
return false
}
if c.sanitize != nil && c.sanitize.inSanitizerDir() {
return true
}
return c.installer.inData()
return c.installer.inSanitizerDir()
}
func (c *Module) HostToolPath() android.OptionalPath {

View file

@ -32,6 +32,7 @@ type installLocation int
const (
InstallInSystem installLocation = 0
InstallInData = iota
InstallInSanitizerDir = iota
)
func NewBaseInstaller(dir, dir64 string, location installLocation) *baseInstaller {
@ -78,6 +79,10 @@ func (installer *baseInstaller) inData() bool {
return installer.location == InstallInData
}
func (installer *baseInstaller) inSanitizerDir() bool {
return installer.location == InstallInSanitizerDir
}
func (installer *baseInstaller) hostToolPath() android.OptionalPath {
return android.OptionalPath{}
}

View file

@ -349,8 +349,8 @@ func (library *libraryDecorator) getLibName(ctx ModuleContext) string {
func (library *libraryDecorator) linkerInit(ctx BaseModuleContext) {
location := InstallInSystem
if library.sanitize.inData() {
location = InstallInData
if library.sanitize.inSanitizerDir() {
location = InstallInSanitizerDir
}
library.baseInstaller.location = location

View file

@ -97,7 +97,7 @@ type SanitizeProperties struct {
SanitizerEnabled bool `blueprint:"mutated"`
SanitizeDep bool `blueprint:"mutated"`
InData bool `blueprint:"mutated"`
InSanitizerDir bool `blueprint:"mutated"`
}
type sanitize struct {
@ -383,8 +383,8 @@ func (sanitize *sanitize) flags(ctx ModuleContext, flags Flags) Flags {
return flags
}
func (sanitize *sanitize) inData() bool {
return sanitize.Properties.InData
func (sanitize *sanitize) inSanitizerDir() bool {
return sanitize.Properties.InSanitizerDir
}
func (sanitize *sanitize) Sanitizer(t sanitizerType) bool {
@ -447,7 +447,7 @@ func sanitizerMutator(t sanitizerType) func(android.BottomUpMutatorContext) {
modules[0].(*Module).sanitize.Properties.SanitizeDep = false
modules[1].(*Module).sanitize.Properties.SanitizeDep = false
if mctx.Device() {
modules[1].(*Module).sanitize.Properties.InData = true
modules[1].(*Module).sanitize.Properties.InSanitizerDir = true
} else {
modules[0].(*Module).Properties.PreventInstall = true
}