[PATCH] ASan Linux MIPS32 support

Kostya Serebryany kcc at google.com
Mon Jun 3 00:48:58 PDT 2013


Hi Jyun-Yan,

The patches look almost perfect, but may I ask you to send them using
http://llvm.org/docs/Phabricator.html ?
(This is not a requirement in the LLVM land, but is our strong preference
for the sanitizer code)
One thing that I noticed is __attribute__((aligned(16))), for which we
prefer to use the ALIGNED macro from
sanitizer_common/sanitizer_internal_defs.h.

The main question about the MIPS support is whether you are planing to have
a public build bot running asan tests?
If not, there is a good chance that we will be breaking it every other
week.

Thanks!

--kcc



On Mon, Jun 3, 2013 at 8:44 AM, Jyun-Yan You <jyyou at cs.nctu.edu.tw> wrote:

> These patches add AddressSanitizer Linux MIPS32 support.
> The shadow offset is calculated from TASK_UNMAPPED_BASE of Linux MIPS32.
> The LLVM patch also makes the poison value use target endian instead of
> host.
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130603/c87e6947/attachment.html>


More information about the llvm-commits mailing list