[PATCH] D26201: [ELF/GC] Fix pending references to garbage collected sections
Michael Spencer via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 1 11:10:58 PDT 2016
Bigcheese added inline comments.
================
Comment at: test/ELF/gc-debuginfo-tls.s:19-28
+ .file 1 "patatino.cpp"
+ .globl main
+main:
+ .loc 1 2 0
+ .cfi_startproc
+ .cfi_endproc
+ .cfi_startproc
----------------
davide wrote:
> Bigcheese wrote:
> > This is a pretty obfuscated way to get a TLS symbol to a gced section. I think this can be simplified to something like:
> >
> > ```
> > .section .tbss,"awT", at nobits
> > patatino:
> > .long 0
> > .section .noalloc,""
> > .quad patatino
> > ```
> This is reduced from the original testcase. But sure, I can add another test case if you want.
Sure, just add a comment to the .cfi* one saying that it was reduced from a testcase and that the debug info directives are generating the reference.
https://reviews.llvm.org/D26201
More information about the llvm-commits
mailing list