[PATCH] D61510: [SanitizerCoverage] Use different module ctor names for trace-pc-guard and inline-8bit-counters

[Fangrui Song via Phabricator via llvm-commits]

MaskRay created this revision.
MaskRay added reviewers: kcc, morehouse, vitalybuka.
Herald added subscribers: llvm-commits, eraman.
Herald added a project: LLVM.

Fixes the main issue in PR41693

When both modes are used, two functions are created:
`sancov.module_ctor`, `sancov.module_ctor.$LastUnique`, where
$LastUnique is a number (current LastUnique counter) that may be
different in different translation units. After linking, there will be
duplicate module ctors.


Repository:
  rL LLVM

https://reviews.llvm.org/D61510

Files:
  lib/Transforms/Instrumentation/SanitizerCoverage.cpp
  test/Instrumentation/SanitizerCoverage/trace-pc-guard-comdat.ll
  test/Instrumentation/SanitizerCoverage/trace-pc-guard-inline-8bit-counters.ll
  test/Instrumentation/SanitizerCoverage/trace-pc-guard-nocomdat.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D61510.198002.patch
Type: text/x-patch
Size: 5540 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190503/2263c23d/attachment.bin>