[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