am 1f134dd0: Merge "Add support for ASAN in host modules."

* commit '1f134dd06cd12f032c833b9a43b476a9557fa8da':
  Add support for ASAN in host modules.

Merge "Add support for ASAN in host modules."

Add support for ASAN in host modules.

Change-Id: Id9ed83499e0eb995f06e845c3153a50677d768b8

am ed8206db: Merge "Remove unnecessary 10.8-specific build rules"

* commit 'ed8206dba48e182f786ec677d56586ffaf973250':
  Remove unnecessary 10.8-specific build rules

Merge "Remove unnecessary 10.8-specific build rules"

am 28c2d3b9: Merge "Fix breakage for x86_64 static executables."

* commit '28c2d3b9852550bc9ae150e2725e08885f1ade73':
  Fix breakage for x86_64 static executables.

Merge "Fix breakage for x86_64 static executables."

Remove unnecessary 10.8-specific build rules

We no longer need gcc for host builds, since those all run through clang. This
header include, however, triggers errors about SSE intrinsics by replacing
the more relevant include dirs that we should be using.

Change-Id: I26a949f0109de8e6e2d1f09cb8127be927549cc4

Fix breakage for x86_64 static executables.

Yikes. Don't know how this slipped through code review.

I had actually mentioned a need for cleanup in this part of the build
system earlier, since the amount of duplication between
transform-o-to-* for each arch means we might fix things incorrectly
in one of them (as I've just shown). Similarly, code reviewers are
likely to skim each one after the first if they all look close enough
(which is presumably what happened here).

Change-Id: I9b85914510f0b114485021deb97f42740712aae5

am 26707ca8: Merge "Use libstc++ for the name of bionic\'s STL."

* commit '26707ca8bb29b62252dd9d3d346f731f4fad2806':
  Use libstc++ for the name of bionic's STL.

Merge "Use libstc++ for the name of bionic's STL."

am fc82be6e: Merge "Add LPAE to dex2oat ISA feature list."

* commit 'fc82be6e776acfc3d784e61e2026d65417285495':
  Add LPAE to dex2oat ISA feature list.

Use libstc++ for the name of bionic's STL.

It is libstdc++.so, after all, and the naming makes sense for the host
this way (since it also uses libstdc++).

Change-Id: If37ffa015f7967a928ea47a290363d7696c4ce35

Merge "Add LPAE to dex2oat ISA feature list."

am 4e3e9ffb: Merge "Fix Java detection on some Linux distributions"

* commit '4e3e9ffbf314500d303724aefbddf1b89094cfd8':
  Fix Java detection on some Linux distributions

Merge "Fix Java detection on some Linux distributions"

am 07b212ac: Merge "Move definition of -D__ARM_FEATURE_LPAE=1 cflag to top-level."

* commit '07b212ac59f0018a01ad3262caa0b96b9aff30a1':
  Move definition of -D__ARM_FEATURE_LPAE=1 cflag to top-level.

Add LPAE to dex2oat ISA feature list.

Bug: 17993736

Change-Id: Ibdf0df017a582d7d807dff17f0392dbf28475e02

Merge "Move definition of -D__ARM_FEATURE_LPAE=1 cflag to top-level."

am f2bdac77: Merge "Never use stlport on the host."

* commit 'f2bdac77b1d99abb190253f386cc6ffe4ffbeaca':
  Never use stlport on the host.

Merge "Never use stlport on the host."

Never use stlport on the host.

LOCAL_CXX_STL := stlport should just use the default STL on the host.

Change-Id: Iede1b2d8884e237a10e000e5835addb768c8f78d

Fix Java detection on some Linux distributions

On some Linux distributions (spotted here on OpenMandriva Lx, but I'm
pretty sure some others do the same thing), "which javac" returns
/usr/bin/javac, which is a symlink to "../../etc/alternatives/javac",
which in turn points at whatever the JDK the user picked as his default.

Given "../../etc/alternatives/javac" is a relative symlink, the next
iteration of LSLINE=$(ls -l "$JAVAC") fails (no ../../etc/alternatives/java
relative to the build directory), causing tools.jar not to be found.

Using realpath and readlink where possible should work in all cases.

Change-Id: Ic60ac84a5b263dc1c1f2960092a7549d1024ed2e
Signed-off-by: Bernhard Rosenkraenzer <Bernhard.Rosenkranzer@linaro.org>

Move definition of -D__ARM_FEATURE_LPAE=1 cflag to top-level.

LPAE indicates better instructions can be used when atomicity guarantees are
needed. However, LPAE's presence isn't advertised by clang/GCC. We fake an
ARM feature to advertise its presence on architectures where it is.
Also, add a TODO documenting that cortex-a15 is not the correct CPU variant
for krait.

Change-Id: I02a1248025c32d94eca0bc8a249dc524f1ac9c36

am c131ec09: Merge "Link compiler-rt in the correct order."

* commit 'c131ec09d9ffb92473c4162c92c0646d6e731b0c':
  Link compiler-rt in the correct order.

Merge "Link compiler-rt in the correct order."

Link compiler-rt in the correct order.

The compiler run-time library should always be the _last_ thing linked
when building static executables. This was being done correctly for
libgcc, but not when using compiler-rt.

Change-Id: I0689dc35f55caad2fe74c0cbb4cbe3008ded349a

am 165fdf42: Merge "Remove "-mstackrealign" option from all x86 builds."

* commit '165fdf42042d7c13d9f7866b3a3eff841e08dad2':
  Remove "-mstackrealign" option from all x86 builds.

Merge "Remove "-mstackrealign" option from all x86 builds."

Remove "-mstackrealign" option from all x86 builds.

For ndk docs change, please refer to:
https://android-review.googlesource.com/#/c/110100/

Change-Id: I8428e7a979eb02441066aeeee43ce693d4d0dc8d
Signed-off-by: Alexander Ivchenko <alexander.ivchenko@intel.com>

am f1794738: Merge "Introduce per-product per-module dex-preopt config"

* commit 'f1794738b178c22bd7f05348f2a393cdec23ade4':
  Introduce per-product per-module dex-preopt config

Merge "Introduce per-product per-module dex-preopt config"

am fb68db08: Merge "Docs: Making source.android.com headers match those of developer.android.com"

* commit 'fb68db08000e2d22461edbfa663e298c672476c0':
  Docs: Making source.android.com headers match those of developer.android.com

Merge "Docs: Making source.android.com headers match those of developer.android.com"

am 29373be8: Merge "Add -fPIC as default Clang mips ASFLAGS."

* commit '29373be81de4d6e8817434b146c395b2a28e4628':
  Add -fPIC as default Clang mips ASFLAGS.

Merge "Add -fPIC as default Clang mips ASFLAGS."

am a01fa4f8: Merge "LOCAL_NATIVE_COVERAGE should imply -O0."

* commit 'a01fa4f8a369580a888317d0ea7f3507f4dcc828':
  LOCAL_NATIVE_COVERAGE should imply -O0.

Merge "LOCAL_NATIVE_COVERAGE should imply -O0."

LOCAL_NATIVE_COVERAGE should imply -O0.

Optimization can lead to incorrect coverage results.

Change-Id: I8ade9ab246e292d26ac18821baf9970e29dc1a1f

Add -fPIC as default Clang mips ASFLAGS.

For mips target, gcc passes -KPIC to assembler by default,
but clang passes -KPIC only if -fPIC or -fpic is given.

BUG: 17895505
Change-Id: I7803adaf8ad7b40c6473872b67f4e0b4c502b38a

Introduce per-product per-module dex-preopt config

- Added LOCAL_DEX_PREOPT_FLAGS to pass extra flags to dex2oat.
- Added macro add-product-dex-preopt-module-config to specify almost
  arbitrary dex-preopt config/flags to modules in product
  configuration:
  $(call \
  add-product-dex-preopt-module-config,<module_name_list>,<config_or_flags>)
  How <config_or_flags> is interpreted is decided by
  dex_preopt_odex_install.mk and dex2oat. For now if it's "disable" we
  disable dexpreopt for the given modules; otherwise pass it to dex2oat as
  command line flags.
- If there are multiple configs for the same module in the product
  inheritance, the first takes precedence.
- Added PRODUCT_DEX_PREOPT_DEFAULT_FLAGS so you can specify default
  dex2oat flags in product configuration.
- Added PRODUCT_DEX_PREOPT_BOOT_FLAGS to specify flags of building boot.oat.

Bug: 17791867
Change-Id: I1b2955e8e51039e94d1ff43a3265a8d03598632c

am 8330c4c5: Merge "Apply LOCAL_CXX_STL to also prebuilts."

* commit '8330c4c5e6f8bb8b64e1b45ceed603b49853a086':
  Apply LOCAL_CXX_STL to also prebuilts.

Merge "Apply LOCAL_CXX_STL to also prebuilts."

Apply LOCAL_CXX_STL to also prebuilts.

Because LOCAL_CXX_STL modifies a module's required shared libaries,
we need this for also prebuilt shared libraries and executables.

Change-Id: I418c26143999a613c40aadf990f131b123e0ac3d

am 9bf56c5e: Merge "Fix typo."

* commit '9bf56c5e911856bc55693f5723a53271147e583e':
  Fix typo.

Merge "Fix typo."

Fix typo.

Change-Id: I738ad878a2961d118b5a89bf7d434606df2fad92

am 4af753c1: Merge "Make which dex2oat version to use configurable."

* commit '4af753c1d1e770419314108ad795b97d17b11380':
  Make which dex2oat version to use configurable.

Merge "Make which dex2oat version to use configurable."

Make which dex2oat version to use configurable.

Change-Id: Ic13cf6f9dfb117193a5101781945c501d138955c

am 642c81ea: Merge "Stash original values of CC and CXX."

* commit '642c81ea4559836236a38123feec3d8e51d4b9c2':
  Stash original values of CC and CXX.

am b6361968: Merge "Remove unused my_compiler_dependencies."

* commit 'b6361968f86a1e96e1cdf5451d62ea35541de1e3':
  Remove unused my_compiler_dependencies.

Merge "Stash original values of CC and CXX."

Merge "Remove unused my_compiler_dependencies."

Remove unused my_compiler_dependencies.

my_compiler_dependencies was never assigned to, but the way it was
included in the rules prevented the user from being able to use | in
LOCAL_ADDITIONAL_DEPENDENCIES. Since it is unneeded, just remove it.

Change-Id: I74bb59e81b97756296060eea5b7a42909be50130

Stash original values of CC and CXX.

This way we can still access the real compiler as opposed to a wrapped
one.

Change-Id: I17ba30416a3eb8eda056c67faaa8b5957e70caea

Docs: Making source.android.com headers match those of developer.android.com

Bug: 17814801
Change-Id: I128a76b32b0d92118f621062c62d72460313f4d6

am 9979ae43: Merge "Remove HAVE_(GNU|BSD)_QSORT_R from AndroidConfig.h."

* commit '9979ae4370e7acd4b305fddeca3c24495214a0cf':
  Remove HAVE_(GNU|BSD)_QSORT_R from AndroidConfig.h.

Merge "Remove HAVE_(GNU|BSD)_QSORT_R from AndroidConfig.h."

Remove HAVE_(GNU|BSD)_QSORT_R from AndroidConfig.h.

Change-Id: If2018529a6c067b63e1a59c16c911d63ec2ebbf4

am 35dd358c: Merge "Update prebuilt RS clcore location."

* commit '35dd358cc4fccb062bd90cb1f9404a582b4facd4':
  Update prebuilt RS clcore location.

Merge "Update prebuilt RS clcore location."

am 09cdd500: Merge "Clear all LOCAL_CLANG_*FLAGS*."

* commit '09cdd5000caed13a135d3ff62972e002904208c3':
  Clear all LOCAL_CLANG_*FLAGS*.

Update prebuilt RS clcore location.

Change-Id: Iebf255d7cc093b74e08b1f1378d9b397fd9dd3b9

Merge "Clear all LOCAL_CLANG_*FLAGS*."

Clear all LOCAL_CLANG_*FLAGS*.

Although only some of these flags will be used for some
targets or hosts, they must all be cleared to avoid carrying
over from one project to another.

BUG: 17677366
Change-Id: I6d26fa7e5bf2ff11758728810d4874f3759aebe7

am 326363b8: Merge "java_alternative_checked_module takes precedence."

* commit '326363b82368efe5a99df558588d5b2f802fe604':
  java_alternative_checked_module takes precedence.

Merge "java_alternative_checked_module takes precedence."

java_alternative_checked_module takes precedence.

java_alternative_checked_module takes precedence over
LOCAL_BUILT_MODULE.

Change-Id: I84f0d65dbc05f4686817aa835d003eb0101af146

am 8d856002: Merge "Don\'t modify LOCAL_CHECKED_MODULE."

* commit '8d8560026a75aaa9e4abe7190a1dab3c4732767d':
  Don't modify LOCAL_CHECKED_MODULE.

Merge "Don't modify LOCAL_CHECKED_MODULE."

Don't modify LOCAL_CHECKED_MODULE.

Use a temporary variable my_checked_module instead,
so that we don't override the 2nd_arch's checked module with the
1st_arch's in multilib build.
Note that by default we checkbuild 2nd_arch only for host modules,
but not for target modules.
We enable multlib for target modules by default, it would take too
much time to checkbuild for both archs. For 32-bit modules actually we
can checkbuild a 32-bit product.

Change-Id: I0a5ba75699225b7641442673483731a8fd360d61

am 00bd658e: Merge "Remove obsolete libffi"

* commit '00bd658ec9436a5ee65077f1d25ccbcc28933bc8':
  Remove obsolete libffi

Merge "Remove obsolete libffi"

Remove obsolete libffi

Change-Id: I092e643be8bde9dbf00dd0d3314275e1ac3db409

am 9d5f5d0e: Merge "Add domains for goldfish services."

* commit '9d5f5d0e49ee3000493af97acd9e62cb08f99ba7':
  Add domains for goldfish services.

Merge "Add domains for goldfish services."

am 4a8d5c1a: Merge "Add local Clang+target specific flags."

* commit '4a8d5c1aaff33a2304850ab66221e0bd4f83003f':
  Add local Clang+target specific flags.

Merge "Add local Clang+target specific flags."

Add local Clang+target specific flags.

BUG: 17677366
Change-Id: I75522fb56fdf4d27ea1f5f0ae15866b13ae6e206

am 91adf75c: Merge "Fix detection of C++ STL for tests."

* commit '91adf75c3e0f13bab71e573ec4e3731699f7136e':
  Fix detection of C++ STL for tests.

Merge "Fix detection of C++ STL for tests."

Fix detection of C++ STL for tests.

Target tests weren't approrpiately using the libc++ gtest for
libc++_static, and the hosts tests were still using the old check.

Change-Id: I13813d5f09673b144b2dfead93eb81cb4bae0e34

am 670018fd: Merge "Remove unnecessary CLANG_CONFIG_EXTRA_*_C_INCLUDES variables."

* commit '670018fdaed33a6872925e60fd95c5f401e45444':
  Remove unnecessary CLANG_CONFIG_EXTRA_*_C_INCLUDES variables.

am 3cf20570: Merge "Use new variable LLVM_PREBUILTS_VERSION to simplify path construction."

* commit '3cf2057079df33e914640c3fe2b1ef7a10495064':
  Use new variable LLVM_PREBUILTS_VERSION to simplify path construction.

Merge "Remove unnecessary CLANG_CONFIG_EXTRA_*_C_INCLUDES variables."

Merge "Use new variable LLVM_PREBUILTS_VERSION to simplify path construction."

am 577184aa: Merge "Test __GLIBC__ is defined before using its value."

* commit '577184aa86eefc4b7b177eeafcd8209028d86b2d':
  Test __GLIBC__ is defined before using its value.

Merge "Test __GLIBC__ is defined before using its value."

Remove unnecessary CLANG_CONFIG_EXTRA_*_C_INCLUDES variables.

These aren't needed now that we only use the compiler/headers that exist in
the prebuilts/clang directory.

Change-Id: I9978efb10815e92577d45629db324e0a5094f880

Test __GLIBC__ is defined before using its value.

Avoids -Wundef warning.

Change-Id: I52d8223500fe31cdf7023e32e96df75e33eb2f7e

Use new variable LLVM_PREBUILTS_VERSION to simplify path construction.

This removes the hard-coded '3.5' in the various paths and makes switching to
a new toolchain easier from the command line (m LLVM_PREBUILTS_VERSION=3.6).

Change-Id: I46b10eb2fc177a03528de9c7b433f8647f632081

am 33029248: Merge "Add option for generating coverage info."

* commit '330292480272aed3d7663ae7a7668675c742b2e0':
  Add option for generating coverage info.

Merge "Add option for generating coverage info."

Add option for generating coverage info.

To enable building with coverage, the environment variable
NATIVE_COVERAGE must be set to true.

Set `LOCAL_NATIVE_COVERAGE := true` to generate coverage information for
a given component.

This is currently not supported for clang (b/17574078, b/17583330).

If static library A is included in a binary B (dynamic or static
executable, or shared library), and A is built with coverage
information, B is required to link with libgcov.a. Since the make does
not offer a good way to track this dependency, link libgcov.a even if
LOCAL_NATIVE_COVERAGE is not set (but still guarded by NATIVE_COVERAGE).
This ensures that all of the libgcov dependencies will always be
resolved, and causes no change in the resulting binary if coverage is
not used.

Bug: 10134489
Change-Id: Id5a19f2c215e4be80e6eae27ecc19b582f2f6813

am cb0755f8: Merge "Remove unused HAVE_OFF64_T."

* commit 'cb0755f8c92c252331614bee7bceeb897357e0f1':
  Remove unused HAVE_OFF64_T.

Merge "Remove unused HAVE_OFF64_T."

Remove unused HAVE_OFF64_T.

Change-Id: I24acf64efcd0fd84d5dda342d4c4293fa59fae1a

am 88c7544e: Merge "Remove the unused IPC macros."

* commit '88c7544e711e70273b3ab29251ba2ff359a7ea1a':
  Remove the unused IPC macros.

Merge "Remove the unused IPC macros."