diff options
| author | George Burgess IV <gbiv@google.com> | 2019-05-24 13:45:36 -0700 |
|---|---|---|
| committer | George Burgess IV <gbiv@google.com> | 2019-05-24 13:51:12 -0700 |
| commit | 373cb301ad226ab33fdd65212ca0c5c78542f504 (patch) | |
| tree | cff9198373e1431cc5265c723bc4bf9fbc2e4581 /libunwindstack/tests/MemoryFileTest.cpp | |
| parent | 3e48b66578fb1fbfc45d3a58adf8e07bf378bc6a (diff) | |
memunreachable_test: don't use `write` to escape addresses
A recent change
(https://android-review.googlesource.com/c/platform/bionic/+/961601)
caused this test to fail. This change allowed LLVM to optimize what was
previously `__write_chk` into a standard call to `write`.
LLVM knows things about `write`. In particular, it knows that `write`
doesn't capture its buffer param, which `Ref` assumes isn't possible.
LLVM can't reason much about `volatile` operations; storing this to a
`volatile` ptr should be good enough. If not, there are a few other
tricks we can play here.
Bug: 133479661
Test: `atest
memunreachable_test:memunreachable_test.MemunreachableTest#stack --
--abi x86` on internal-master
Change-Id: I905e510210cafdf83fa1b50f2f090d4f8b9653da
Diffstat (limited to 'libunwindstack/tests/MemoryFileTest.cpp')
0 files changed, 0 insertions, 0 deletions