LineageOS/android_development
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fix exclusion of builtin types in linked dump.

Browse Source 
Builtin types do not need an 'IsPresentInExportedHeaders' check while
linking.

Bug: 63865902

Test: mm -j64 in external/libjpeg-turbo, resulting linked abi dump has
      builtin types (when used with built version on header-abi-linker)

Change-Id: I7816c6522b8a4873ebcdf6c325a6e224eab48de7
This commit is contained in:
Jayant Chowdhary
2017-11-30 12:05:25 -08:00
parent 6a233fe18a
commit d39c93143f
1 changed files with 0 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
vndk/tools/header-checker/header-abi-util/src/ir_representation_protobuf.cpp
View File

@@ -218,9 +218,6 @@ void ProtobufTextFormatToIRReader::ReadBuiltinTypes(
ReadTypeInfo(builtin_type_protobuf.type_info(), &builtin_type_ir); ReadTypeInfo(builtin_type_protobuf.type_info(), &builtin_type_ir);
builtin_type_ir.SetSignedness(builtin_type_protobuf.is_unsigned()); builtin_type_ir.SetSignedness(builtin_type_protobuf.is_unsigned());
builtin_type_ir.SetIntegralType(builtin_type_protobuf.is_integral()); builtin_type_ir.SetIntegralType(builtin_type_protobuf.is_integral());
if (!IsPresentInExportedHeaders(builtin_type_ir, exported_headers_)) {
continue;
}
builtin_types_.insert( builtin_types_.insert(
{builtin_type_ir.GetLinkerSetKey(), std::move(builtin_type_ir)}); {builtin_type_ir.GetLinkerSetKey(), std::move(builtin_type_ir)});
} }