[PATCH] D97755: [IRSymTab] Set FB_used on llvm.compiler.used symbols

[Teresa Johnson via Phabricator via llvm-commits]

tejohnson added inline comments.


================
Comment at: clang/test/CodeGen/thinlto-inline-asm2.c:10
+
+//--- a.c
+const char *ref() {
----------------
Is this other file needed for the test?


================
Comment at: llvm/test/ThinLTO/X86/asm.ll:11
+; NM-NOT: {{.}}
+; NM:     ---------------- T ref
+; NM-NOT: {{.}}
----------------
MaskRay wrote:
> tejohnson wrote:
> > What is the behavior here without this patch?
> This is not changed. `ff_h264_cabac_tables` is not in the IR symbol table.
Same question here - is this file needed to test the handling of the llvm.compiler.used in the other file?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D97755