[PATCH] D128863: Add switch to use "source_file_name" instead of Module ID for globally promoted local

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added a comment.

Can you simplify the test (see comment there) and also update the summary to reflect that we usually use a module hash (not a Module ID hash) - what's changing in your case is the hash of the module's bitcode.



================
Comment at: llvm/test/ThinLTO/X86/promote-local-name.ll:20
+
+; The copy in %t1.bc should not be exported/promoted/renamed
+; RUN: llvm-lto -use-source-filename-for-promoted-locals -thinlto-action=promote %t1.bc -thinlto-index=%t3.bc -o - | llvm-dis -o - | FileCheck %s --check-prefix=NOEXPORTSTATIC
----------------
I don't think there is a need for this testing here (this is behavior tested elsewhere). You should be able to remove promote-local-name-1.ll completely from this test.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D128863