[PATCH] D62701: [PDB] Copy inlinee lines records into the PDB

[Reid Kleckner via Phabricator via llvm-commits]

rnk marked 2 inline comments as done.
rnk added inline comments.


================
Comment at: lld/test/COFF/pdb-inlinees-extrafiles.s:6
+
+# The assembly was hand written to model the following C code. As of this
+# writing, clang does not emit extra files for inlinees, so it had to be hand
----------------
aganea wrote:
> Why not:
> ```
> $ cl /Z7 /c /O2 t.c
> $ obj2yaml t.obj >t.yaml
> ```
> Which changes the test to:
> ```
> # REQUIRES: x86
> # RUN: yaml2obj %s -o=%t.obj
> # RUN: lld-link -entry:main -nodefaultlib %t.obj -out:%t.exe -pdb:%t.pdb -debug
> # RUN: llvm-pdbutil dump -il %t.pdb | FileCheck %s
> ```
> I think it should be made clear/easy to re-generate the tests, if we want to change it, or if someone wants to duplicate it.
> 
Basically it boils down to, which do we think is a more useful test format, YAML or assembly? My preference is for assembly, and I'd like to replace a lot of the .test YAML inputs with .s inputs. Maybe that's unique to me, but it mirrors the direction the ELF linker took, where they moved away from YAML object input tests to assembly tests.

I can't generate assembly from MSVC, so I started with clang assembly output, and modified it to exercise the corner case in question.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D62701/new/

https://reviews.llvm.org/D62701