[PATCH] D19236: Add DITypeIndex for CodeView and emit it for locals

[Amjad Aboud via llvm-commits]

aaboud added a subscriber: aaboud.
aaboud added a comment.

I am a little confused.

1. Are you trying to replace DIBasicType?
2. Will Clang be generating DITypeIndex for Windows target (COFF debug info) and DIBasicType/DICompositeType/DIDerivedType/DISubroutineType for Linux target (DWARF debug info)?
3. You did not explain how you will represent the compound types using this DITypeIndex! 3.a. Who will set the index field in DITypeIndex for compound types? can clang FE do that? 3.b. Will CodeView emitter emit that index directly? what will happen if we do not want to emit some of the types, do not we need to rearrange the indices? 3.c. Will this work also for emitting a pdb file? can the compiler set these indices also when emitting types to pdb file instead of COFF object file?

I would appreciate if you can answer these questions.
Thanks,
Amjad


http://reviews.llvm.org/D19236