[all-commits] [llvm/llvm-project] 26fd95: [GWP-ASan] 32-bit test pointers, allow multi-init ...

Mitch Phillips via All-commits all-commits at lists.llvm.org
Mon Dec 9 08:20:51 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 26fd95680bcf96870fbe9187aaa7e460709ce487
      https://github.com/llvm/llvm-project/commit/26fd95680bcf96870fbe9187aaa7e460709ce487
  Author: Mitch Phillips <31459023+hctim at users.noreply.github.com>
  Date:   2019-12-09 (Mon, 09 Dec 2019)

  Changed paths:
    M compiler-rt/lib/gwp_asan/guarded_pool_allocator.cpp
    M compiler-rt/lib/gwp_asan/tests/compression.cpp

  Log Message:
  -----------
  [GWP-ASan] 32-bit test pointers, allow multi-init for test.

Summary:
GWP-ASan test currently fail on 32-bit platforms, as some of the pointers are
larger than `uintptr_t` on 32-bit platforms. Fix up all those instances.

Also add an uncompress varint test where the result is an underflow.

Furthermore, allow multi-init for testing. Each gtest when running
`check-gwp_asan` apparently runs in its own instance, but when integrating
these tests into Android, this behaviour isn't the same. We remove the
global multi-init check here, to allow for testing to work elsewhere, and we're
not really worried about multi-init anyway as it's part of our contract with
the allocator.

Reviewers: eugenis, vlad.tsyrklevich

Reviewed By: eugenis

Subscribers: #sanitizers, llvm-commits, pcc

Tags: #sanitizers, #llvm

Differential Revision: https://reviews.llvm.org/D71121




More information about the All-commits mailing list