[all-commits] [llvm/llvm-project] 47cc6d: Re-land [Debug][CodeView] Emit fully qualified nam...

[Reid Kleckner via All-commits]

  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 47cc6db928d063d96e11e70c196bd5601b2bdd06
      https://github.com/llvm/llvm-project/commit/47cc6db928d063d96e11e70c196bd5601b2bdd06
  Author: Reid Kleckner <rnk at google.com>
  Date:   2020-05-18 (Mon, 18 May 2020)

  Changed paths:
    M lldb/test/Shell/SymbolFile/PDB/variables.test
    M llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
    M llvm/lib/CodeGen/AsmPrinter/CodeViewDebug.h
    M llvm/test/DebugInfo/COFF/global-constants.ll
    M llvm/test/DebugInfo/COFF/global_visibility.ll
    M llvm/test/DebugInfo/COFF/globals.ll
    M llvm/test/DebugInfo/COFF/types-array-unsized.ll
    A llvm/test/DebugInfo/COFF/udts-fixpoint.ll

  Log Message:
  -----------
  Re-land [Debug][CodeView] Emit fully qualified names for globals

This reverts commit 525a591f0f48b9d54018bf5245f2abee09c9c1c8.

Fixed an issue with pointers to members based on typedefs. In this case,
LLVM would emit a second UDT. I fixed it by not passing the class type
to getTypeIndex when the base type is not a function type. lowerType
only uses the class type for direct function types. This suggests if we
have a PMF with a function typedef, there may be an issue, but that can
be solved separately.