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

[Adrian Prantl via llvm-commits]

aprantl added a comment.

The code itself looks fine to me.


================
Comment at: lib/Bitcode/Writer/ValueEnumerator.cpp:351
@@ -350,2 +350,3 @@
     for (const auto &I : MDs)
-      EnumerateMetadata(&GV, I.second);
+      // FIXME: Pass GV to EnumerateMetadata and arrange for the bitcode writer
+      // to write metadata to the global variable's own metadata block.
----------------
I'm just curious: Is this being addressed in one of the follow-up reviews? If not, can you please file a PR about it?


http://reviews.llvm.org/D21052