[PATCH] D21053: IR: New representation for CFI and virtual call optimization pass metadata.

Peter Collingbourne via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 21 18:31:50 PDT 2016


pcc marked an inline comment as done.

================
Comment at: lib/Transforms/IPO/CrossDSOCFI.cpp:77
@@ -78,1 +76,3 @@
+/// Extracts a numeric type identifier from an MDNode containing type metadata.
+ConstantInt *CrossDSOCFI::extractNumericTypeId(GlobalObject &GO, MDNode *MD) {
   // This check excludes vtables for classes inside anonymous namespaces.
----------------
eugenis wrote:
> GlobalObject argument is now unused.
Removed

================
Comment at: lib/Transforms/IPO/LowerTypeTests.cpp:960
@@ +959,3 @@
+      AddGlobalToEqClass(F);
+    for (GlobalVariable &GV : M->globals())
+      AddGlobalToEqClass(GV);
----------------
eugenis wrote:
> I'm surprised we don't have Module::global_objects() yet. It sounds like a great idea.
Done in D21580; this patch now depends on it.


http://reviews.llvm.org/D21053





More information about the llvm-commits mailing list