llvm/test/Instrumentation
Duncan P. N. Exon Smith a5ae7c1c9f DI: Require subprogram definitions to be distinct
As a follow-up to r246098, require `DISubprogram` definitions
(`isDefinition: true`) to be 'distinct'.  Specifically, add an assembler
check, a verifier check, and bitcode upgrading logic to combat testcase
bitrot after the `DIBuilder` change.

While working on the testcases, I realized that
test/Linker/subprogram-linkonce-weak-odr.ll isn't relevant anymore.  Its
purpose was to check for a corner case in PR22792 where two subprogram
definitions match exactly and share the same metadata node.  The new
verifier check, requiring that subprogram definitions are 'distinct',
precludes that possibility.

I updated almost all the IR with the following script:

    git grep -l -E -e '= !DISubprogram\(.* isDefinition: true' |
    grep -v test/Bitcode |
    xargs sed -i '' -e 's/= \(!DISubprogram(.*, isDefinition: true\)/= distinct \1/'

Likely some variant of would work for out-of-tree testcases.

git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@246327 91177308-0d34-0410-b5e6-96231b3b80d8
2015-08-28 20:26:49 +00:00
..
AddressSanitizer DI: Require subprogram definitions to be distinct 2015-08-28 20:26:49 +00:00
BoundsChecking Add a bunch of CHECK missing colons in tests. NFC. 2015-03-14 01:43:57 +00:00
DataFlowSanitizer DI: Require subprogram definitions to be distinct 2015-08-28 20:26:49 +00:00
InstrProfiling Final fix for PR 23499 and IR test case. 2015-05-27 19:34:01 +00:00
MemorySanitizer DI: Require subprogram definitions to be distinct 2015-08-28 20:26:49 +00:00
SanitizerCoverage DI: Require subprogram definitions to be distinct 2015-08-28 20:26:49 +00:00
ThreadSanitizer DI: Require subprogram definitions to be distinct 2015-08-28 20:26:49 +00:00