[PATCH] D50483: [SanitizerCoverage] Add associated metadata to PC guards.

Matt Morehouse via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 10 10:09:09 PDT 2018


morehouse added a subscriber: kcc.
morehouse added a comment.

In https://reviews.llvm.org/D50483#1194694, @eugenis wrote:

> As I understood, the runtime library relies on the fact that when the linker constructs sections for pcguards and for counters, it does that in the same order. This is not guaranteed.


Removed test for this behavior and instead added a test analogous to what we do for `-fsanitize-coverage=inline-8bit-counters,pc-table`.  When @kcc is back, maybe we should discuss changing SanCov to not rely on the same ordering.


https://reviews.llvm.org/D50483





More information about the llvm-commits mailing list