[PATCH] D74364: [GWP-ASan] Update alignment on Android.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 12 15:18:11 PST 2020


morehouse accepted this revision.
morehouse added inline comments.


================
Comment at: compiler-rt/lib/gwp_asan/tests/alignment.cpp:29
+      {1, 8},   {2, 8},   {3, 8},       {4, 8},       {5, 8},   {7, 8},
+      {8, 8},   {9, 16},  {15, 16},     {16, 16},     {17, 24}, {31, 32},
+      {32, 32}, {33, 40}, {4095, 4096}, {4096, 4096},
----------------
Nit:  we technically don't care about alignments >16, so why test for them?  (same for PowerOfTwo test)


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D74364/new/

https://reviews.llvm.org/D74364





More information about the llvm-commits mailing list