[PATCH] D106930: [ThinLTO] Disallow importing for functions with indir branch to block address

Teresa Johnson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 28 09:39:46 PDT 2021


tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.

LGTM with a test fix below.



================
Comment at: llvm/test/ThinLTO/X86/globals-import-blockaddr.ll:11
+; Verify that bar is not imported since it has address-taken block that is target of indirect branch
+; CHECK-NOT: available_externally [1 x i8*]* @bar
 
----------------
This only checks that there isn't an imported copy of bar defined after the imported copy of foo. Either add the same CHECK-NOT prior to the CHECK for foo, or make it a positive sense check for the declare of bar. Probably the latter is cleaner.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D106930



More information about the llvm-commits mailing list