]> granicus.if.org Git - llvm/commit
Fix `FaultMaps` crash when the out streamer is reused
authorYichao Yu <yyc1992@gmail.com>
Tue, 17 Oct 2017 11:44:34 +0000 (11:44 +0000)
committerYichao Yu <yyc1992@gmail.com>
Tue, 17 Oct 2017 11:44:34 +0000 (11:44 +0000)
commitf500b96fb913882fc74166dceb8da6dadb6ed1d1
tree7d911a75479a2f2d78f3864d73aaf78f54b6e5ff
parent0782a95249d46d1facf35b6d6002d6025bf65ebc
Fix `FaultMaps` crash when the out streamer is reused

Summary:
Make sure the map is cleared before processing a new module. Similar to what is done on `StackMaps`.

This issue is similar to D38588, though this time for FaultMaps (on x86) rather than ARM/AArch64. Other than possible mixing of information between modules, the crash is caused by the pointers values in the map that was allocated by the bump pointer allocator that is unwinded when emitting the next file. This issue has been around since 3.8.

This issue is likely much harder to write a test for since AFAICT it requires emitting something much more compilcated (and possibly real code) instead of just some random bytes.

Reviewers: skatkov, sanjoy

Reviewed By: skatkov, sanjoy

Subscribers: sanjoy, aemerson, llvm-commits, kristof.beyls

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

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@315990 91177308-0d34-0410-b5e6-96231b3b80d8
include/llvm/CodeGen/FaultMaps.h
lib/Target/X86/X86AsmPrinter.h