This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG584cb67d2df3: [IRSymTab] Set FB_used on llvm.compiler.used
symbols (authored by MaskRay).
Changed prior to commit:
https://reviews.llvm.org/D97755
MaskRay updated this revision to Diff 327958.
MaskRay edited the summary of this revision.
MaskRay added a comment.
Add llvm-nm checks
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97755/new/
https://reviews.llvm.org/D97755
Files:
clang/test/Cod
tejohnson accepted this revision.
tejohnson added a comment.
This revision is now accepted and ready to land.
lgtm with change suggested below. Thanks!
Comment at: clang/test/CodeGen/thinlto-inline-asm2.c:10
+
+//--- a.c
+const char *ref() {
MaskRay wrote:
> te
MaskRay added inline comments.
Comment at: clang/test/CodeGen/thinlto-inline-asm2.c:10
+
+//--- a.c
+const char *ref() {
tejohnson wrote:
> Is this other file needed for the test?
This is test the limitation.
If we fix the tracking for inline asm references (thi
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: -
MaskRay updated this revision to Diff 327909.
MaskRay added a comment.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
Add a clang/test/CodeGen/ test
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D97755/new/
https://reviews.ll
