[PATCH] D32688: [Coverage] Comdat section name should be same as the variable name in COFF format

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon May 1 10:54:14 PDT 2017


vsk added a reviewer: davidxl.
vsk added a comment.

Could you update some tests in 'test/Instrumentation/InstrProfiling/' to reflect this change? PR23499.ll and linkage.ll are good candidates.

Also, do you know why the change in r256220 ([PGO] Fix another comdat related issue for COFF) was not enough to fix linkage issues when using COFF?


https://reviews.llvm.org/D32688





More information about the llvm-commits mailing list