[PATCH] D107645: [GlobalDCE] Handle relative pointers in VFE (for Swift vtables)

[Kuba (Brecka) Mracek via Phabricator via llvm-commits]

kubamracek added inline comments.


================
Comment at: llvm/lib/Analysis/TypeMetadataUtils.cpp:196
+      if (Operand1TargetGlobal != TopLevelGlobal)
+        return nullptr;
+
----------------
fhahn wrote:
> Could you also add a test case for that?
That's tough without <https://reviews.llvm.org/D108741>, because returning "not found" from this function is currently going to make GlobalDCE be *more aggressive* and remove that slot from the vtable. Writing a test for this behavior is... "weird" because that behavior sounds like a bug (to be fixed with <https://reviews.llvm.org/D108741>).


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D107645/new/

https://reviews.llvm.org/D107645