[PATCH] D21052: IR: Allow metadata attachments on declarations, and fix lazy loaded metadata issue with globals.

[Duncan P. N. Exon Smith via llvm-commits]

Sorry for the delay.  At a high-level, here are my concerns (which you may have already addressed: I haven't looked at the patch yet):

- !dbg attachments on declarations should be disallowed (I suggest the verifier?).
- PGO attachments on declarations should probably also be disallowed?
- Along the same lines, I think having two !dbg (or PGO) attachments on the same function should be disallowed.  I can imagine loosening this restriction in the future (with lots more work) but I think right now it would cause a bug.
- Lazy-loading needs to be fast.

I won't have time to review this in detail for at least a few days, but I'm also happy to defer to others.

> On 2016-Jun-14, at 11:13, Peter Collingbourne <peter at pcc.me.uk> wrote:
> 
> pcc added a comment.
> 
> Ping. Note that http://reviews.llvm.org/D20147 is blocked on this.
> 
> 
> http://reviews.llvm.org/D21052
> 
> 
>