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



================
Comment at: lib/Basic/TargetInfo.cpp:72
+    // For 64-bit Android, alignment is 8 bytes for allocations <= 8 bytes.
+    NewAlign = (Triple.isArch64Bit() || Triple.isArch32Bit()) ? 64 : 0;
+  } else
----------------
Might as well just set `NewAlign = 64;` here.  But not a big deal either way.


Repository:
  rC Clang

https://reviews.llvm.org/D50683



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D50683: [... Pirama Arumuga Nainar via Phabricator via cfe-commits
    • [PATCH] D506... Eli Friedman via Phabricator via cfe-commits
    • [PATCH] D506... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D506... Eli Friedman via Phabricator via cfe-commits
    • [PATCH] D506... Pirama Arumuga Nainar via Phabricator via cfe-commits

Reply via email to