[PATCH] D34514: [mips] Enable IAS by default for Android 64-bit MIPS target (N64)

[Simon Dardis via Phabricator via cfe-commits]

sdardis accepted this revision.
sdardis added a comment.
This revision is now accepted and ready to land.

LGTM with a comment clarity tweak.



================
Comment at: lib/Driver/ToolChains/Gnu.cpp:2322-2323
   case llvm::Triple::mips64el:
-    // Enabled for Debian mips64/mips64el only. Other targets are unable to
-    // distinguish N32 from N64.
-    if (getTriple().getEnvironment() == llvm::Triple::GNUABI64)
+    // Enabled for Debian mips64/mips64el and Android (uses N64 ABI).
+    // Other targets are unable to distinguish N32 from N64.
+    if (getTriple().getEnvironment() == llvm::Triple::GNUABI64 ||
----------------
Enabled for Debian and Android mips64/mipsel, as they can precisely
identify the ABI in use (Debian) or only use N64 for MIPS64 (Android).
Other targets are unable to distinguish N32 from N64.


Repository:
  rL LLVM

https://reviews.llvm.org/D34514