[PATCH] D11180: Fixed debug info generation for function static variables, typedef, and records

[David Blaikie via llvm-commits]

dblaikie added a comment.

It'd be good to split this up as much as possible - it'll make review & commit history easier, etc.

Could you create a separate review just for the "don't omit declarations for unused (fully inlined and optimized away) functions" part that we discussed & you've included here?

After that I think, if I recall our conversation correctly, it might be handy to look at delaying the adding of static variables and local types until the end of the module - and put them in the abstract origin when there is one (rather than always putting them in the concrete definition - and losing them if there is no concrete definition? (that's the current behavior, yes?)).

Then after that, look at trying to keep track of the right scopes, the minimal scopes, etc?


Repository:
  rL LLVM

http://reviews.llvm.org/D11180