diff --git a/llvm/lib/Object/ObjectFile.cpp b/llvm/lib/Object/ObjectFile.cpp --- a/llvm/lib/Object/ObjectFile.cpp +++ b/llvm/lib/Object/ObjectFile.cpp @@ -79,7 +79,7 @@ bool ObjectFile::isSectionBitcode(DataRefImpl Sec) const { Expected NameOrErr = getSectionName(Sec); if (NameOrErr) - return *NameOrErr == ".llvmbc" || *NameOrErr == ".llvm.lto"; + return *NameOrErr == ".llvm.lto"; consumeError(NameOrErr.takeError()); return false; } diff --git a/llvm/test/LTO/X86/Inputs/bcsection.macho.s b/llvm/test/LTO/X86/Inputs/bcsection.macho.s --- a/llvm/test/LTO/X86/Inputs/bcsection.macho.s +++ b/llvm/test/LTO/X86/Inputs/bcsection.macho.s @@ -2,4 +2,4 @@ .asciz "Wrong Section" .section __LLVM,__bitcode -.incbin "bcsection.bc" +.incbin "llvm.lto.section.bc" diff --git a/llvm/test/LTO/X86/Inputs/llvm.lto.section.s b/llvm/test/LTO/X86/Inputs/llvm.lto.section.s new file mode 100644 --- /dev/null +++ b/llvm/test/LTO/X86/Inputs/llvm.lto.section.s @@ -0,0 +1,2 @@ +.section .llvm.lto +.incbin "llvm.lto.section.bc" diff --git a/llvm/test/LTO/X86/bcsection.ll b/llvm/test/LTO/X86/bcsection.ll --- a/llvm/test/LTO/X86/bcsection.ll +++ b/llvm/test/LTO/X86/bcsection.ll @@ -2,23 +2,17 @@ ; RUN: llvm-as -o %t/bcsection.bc %s ; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-pc-win32 -o %t/bcsection.coff.bco %p/Inputs/bcsection.s -; RUN: llvm-nm %t/bcsection.coff.bco | FileCheck %s -; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.coff.o %t/bcsection.coff.bco -; RUN: llvm-nm %t/bcsection.coff.o | FileCheck %s +; RUN: llvm-nm %t/bcsection.coff.bco | FileCheck %s --allow-empty +; RUN: not llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.coff.o %t/bcsection.coff.bco ; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-unknown-linux-gnu -o %t/bcsection.elf.bco %p/Inputs/bcsection.s -; RUN: llvm-nm %t/bcsection.elf.bco | FileCheck %s -; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.elf.o %t/bcsection.elf.bco -; RUN: llvm-nm %t/bcsection.elf.o | FileCheck %s - -; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-apple-darwin11 -o %t/bcsection.macho.bco %p/Inputs/bcsection.macho.s -; RUN: llvm-nm %t/bcsection.macho.bco | FileCheck %s -; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.macho.o %t/bcsection.macho.bco -; RUN: llvm-nm %t/bcsection.macho.o | FileCheck %s +; RUN: llvm-nm %t/bcsection.elf.bco | FileCheck %s --allow-empty +; RUN: not llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.elf.o %t/bcsection.elf.bco target triple = "x86_64-unknown-linux-gnu" -; CHECK: main +;; The .llvmbc section is not intended for use with LTO, so there should be nothing here +; CHECK-NOT: main define i32 @main() { ret i32 0 } diff --git a/llvm/test/LTO/X86/bcsection.ll b/llvm/test/LTO/X86/llvm.lto.section.ll copy from llvm/test/LTO/X86/bcsection.ll copy to llvm/test/LTO/X86/llvm.lto.section.ll --- a/llvm/test/LTO/X86/bcsection.ll +++ b/llvm/test/LTO/X86/llvm.lto.section.ll @@ -1,15 +1,16 @@ ; RUN: rm -rf %t && mkdir -p %t -; RUN: llvm-as -o %t/bcsection.bc %s +; RUN: llvm-as -o %t/llvm.lto.section.bc %s -; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-pc-win32 -o %t/bcsection.coff.bco %p/Inputs/bcsection.s -; RUN: llvm-nm %t/bcsection.coff.bco | FileCheck %s -; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.coff.o %t/bcsection.coff.bco -; RUN: llvm-nm %t/bcsection.coff.o | FileCheck %s +; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-pc-win32 -o %t/llvm.lto.section.coff.bco %p/Inputs/llvm.lto.section.s +; RUN: llvm-nm %t/llvm.lto.section.coff.bco | FileCheck %s +; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/llvm.lto.section.coff.o %t/llvm.lto.section.coff.bco +; RUN: llvm-nm %t/llvm.lto.section.coff.o | FileCheck %s + +; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-unknown-linux-gnu -o %t/llvm.lto.section.elf.bco %p/Inputs/llvm.lto.section.s +; RUN: llvm-nm %t/llvm.lto.section.elf.bco | FileCheck %s +; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/llvm.lto.section.elf.o %t/llvm.lto.section.elf.bco +; RUN: llvm-nm %t/llvm.lto.section.elf.o | FileCheck %s -; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-unknown-linux-gnu -o %t/bcsection.elf.bco %p/Inputs/bcsection.s -; RUN: llvm-nm %t/bcsection.elf.bco | FileCheck %s -; RUN: llvm-lto -exported-symbol=main -exported-symbol=_main -o %t/bcsection.elf.o %t/bcsection.elf.bco -; RUN: llvm-nm %t/bcsection.elf.o | FileCheck %s ; RUN: llvm-mc -I=%t -filetype=obj -triple=x86_64-apple-darwin11 -o %t/bcsection.macho.bco %p/Inputs/bcsection.macho.s ; RUN: llvm-nm %t/bcsection.macho.bco | FileCheck %s diff --git a/llvm/test/tools/gold/X86/fatlto/Inputs/a-LTO.ll b/llvm/test/tools/gold/X86/fatlto/Inputs/a-LTO.ll new file mode 100644 --- /dev/null +++ b/llvm/test/tools/gold/X86/fatlto/Inputs/a-LTO.ll @@ -0,0 +1,29 @@ +; ModuleID = 'a-LTO.bc' +source_filename = "a.c" +target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-unknown-linux-gnu" + +; Function Attrs: noinline nounwind uwtable +define dso_local i32 @a() #0 { +entry: + ret i32 0 +} + +attributes #0 = { noinline nounwind uwtable "frame-pointer"="all" "min-legal-vector-width"="0" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="x86-64" "target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87" "tune-cpu"="generic" } + +!llvm.module.flags = !{!0, !1, !2, !3, !4, !5, !6} +!llvm.ident = !{!7} + +!0 = !{i32 1, !"wchar_size", i32 4} +!1 = !{i32 7, !"PIC Level", i32 2} +!2 = !{i32 7, !"PIE Level", i32 2} +!3 = !{i32 7, !"uwtable", i32 2} +!4 = !{i32 7, !"frame-pointer", i32 2} +!5 = !{i32 1, !"ThinLTO", i32 0} +!6 = !{i32 1, !"EnableSplitLTOUnit", i32 1} +!7 = !{!"clang version 15.0.0 (https://github.com/llvm/llvm-project.git 2288ebbec148ce8000fc2c1f63a4033cd847669e)"} + +^0 = module: (path: "a-LTO.bc", hash: (0, 0, 0, 0, 0)) +^1 = gv: (name: "a", summaries: (function: (module: ^0, flags: (linkage: external, visibility: default, notEligibleToImport: 1, live: 0, dsoLocal: 1, canAutoHide: 0), insts: 1, funcFlags: (readNone: 0, readOnly: 0, noRecurse: 0, returnDoesNotAlias: 0, noInline: 1, alwaysInline: 0, noUnwind: 1, mayThrow: 0, hasUnknownCall: 0, mustBeUnreachable: 0)))) ; guid = 12157170054180749580 +^2 = flags: 8 +^3 = blockcount: 1 diff --git a/llvm/test/tools/gold/X86/fatlto/Inputs/main-LTO.ll b/llvm/test/tools/gold/X86/fatlto/Inputs/main-LTO.ll new file mode 100644 --- /dev/null +++ b/llvm/test/tools/gold/X86/fatlto/Inputs/main-LTO.ll @@ -0,0 +1,36 @@ +; ModuleID = 'main-LTO.bc' +source_filename = "main.c" +target datalayout = "e-m:e-p270:32:32-p271:32:32-p272:64:64-i64:64-f80:128-n8:16:32:64-S128" +target triple = "x86_64-unknown-linux-gnu" + +; Function Attrs: noinline nounwind uwtable +define dso_local i32 @main() #0 { +entry: + %retval = alloca i32, align 4 + store i32 0, ptr %retval, align 4 + %call = call i32 (...) @a() + ret i32 %call +} + +declare i32 @a(...) #1 + +attributes #0 = { noinline nounwind uwtable "frame-pointer"="all" "min-legal-vector-width"="0" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="x86-64" "target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87" "tune-cpu"="generic" } +attributes #1 = { "frame-pointer"="all" "no-trapping-math"="true" "stack-protector-buffer-size"="8" "target-cpu"="x86-64" "target-features"="+cx8,+fxsr,+mmx,+sse,+sse2,+x87" "tune-cpu"="generic" } + +!llvm.module.flags = !{!0, !1, !2, !3, !4, !5, !6} +!llvm.ident = !{!7} + +!0 = !{i32 1, !"wchar_size", i32 4} +!1 = !{i32 7, !"PIC Level", i32 2} +!2 = !{i32 7, !"PIE Level", i32 2} +!3 = !{i32 7, !"uwtable", i32 2} +!4 = !{i32 7, !"frame-pointer", i32 2} +!5 = !{i32 1, !"ThinLTO", i32 0} +!6 = !{i32 1, !"EnableSplitLTOUnit", i32 1} +!7 = !{!"clang version 15.0.0 (https://github.com/llvm/llvm-project.git 2288ebbec148ce8000fc2c1f63a4033cd847669e)"} + +^0 = module: (path: "main-LTO.bc", hash: (0, 0, 0, 0, 0)) +^1 = gv: (name: "a") ; guid = 12157170054180749580 +^2 = gv: (name: "main", summaries: (function: (module: ^0, flags: (linkage: external, visibility: default, notEligibleToImport: 1, live: 0, dsoLocal: 1, canAutoHide: 0), insts: 4, funcFlags: (readNone: 0, readOnly: 0, noRecurse: 0, returnDoesNotAlias: 0, noInline: 1, alwaysInline: 0, noUnwind: 1, mayThrow: 0, hasUnknownCall: 0, mustBeUnreachable: 0), calls: ((callee: ^1))))) ; guid = 15822663052811949562 +^3 = flags: 8 +^4 = blockcount: 1 diff --git a/llvm/test/tools/gold/X86/fatlto/fatlto.test b/llvm/test/tools/gold/X86/fatlto/fatlto.test new file mode 100644 --- /dev/null +++ b/llvm/test/tools/gold/X86/fatlto/fatlto.test @@ -0,0 +1,56 @@ +;; Basic FatLTO tests. + +;; Clean up and initialize test dir +; RUN: rm -rf %t +; RUN: mkdir -p %t + +;; Ensure that input files contain .llvm.lto section +; RUN: llc %p/Inputs/a-LTO.ll --filetype=obj -o %t/a-fatLTO.o +; RUN: llvm-as %p/Inputs/a-LTO.ll -o %t/a-fatLTO.bc +; RUN: llvm-objcopy --add-section=.llvm.lto=%t/a-fatLTO.bc %t/a-fatLTO.o +; RUN: llvm-objcopy --set-section-flags=.llvm.lto=exclude %t/a-fatLTO.o +; RUN: llvm-readobj -S %t/a-fatLTO.o | FileCheck --check-prefix=CHECK-A %s + +; CHECK-A: Name: .llvm.lto + +; RUN: llc %p/Inputs/main-LTO.ll --filetype=obj -o %t/main-fatLTO.o +; RUN: llvm-as %p/Inputs/main-LTO.ll -o %t/main-fatLTO.bc +; RUN: llvm-objcopy --add-section=.llvm.lto=%t/main-fatLTO.bc %t/main-fatLTO.o +; RUN: llvm-objcopy --set-section-flags=.llvm.lto=exclude %t/main-fatLTO.o +; RUN: llvm-readobj -S %t/main-fatLTO.o | FileCheck --check-prefix=CHECK-MAIN %s + +; CHECK-MAIN: Name: .llvm.lto + +; Final executable should not have .llvm.lto section no matter what the target is +; RUN: %gold -plugin %llvmshlibdir/LLVMgold%shlibext -o %t/foo-fatLTO %t/a-fatLTO.o %t/main-fatLTO.o +; RUN: llvm-readobj -S %t/foo-fatLTO | FileCheck --check-prefix=CHECK-LTO-TARGET %s + +;; Check if .llvm.lto section gets aggregated in LTO target +; CHECK-LTO-TARGET-NOT: Name: .llvm.lto + +;; Final executable should not have .llvm.lto section no matter what the target is +; RUN: %gold -o %t/foo-fatNoLTO %t/a-fatLTO.o %/t/main-fatLTO.o +; RUN: llvm-readobj -S %t/foo-fatNoLTO | FileCheck --check-prefix=CHECK-NON-LTO-TARGET %s + +;; Check if .llvm.lto section gets aggregated in non-LTO target +; CHECK-NON-LTO-TARGET-NOT: Name: .llvm.lto + +;; Check if the LTO target executable produced from FatLTO object file is +;; identical to the one produced from LTO modules +; RUN: llvm-as %p/Inputs/a-LTO.ll -o %t/a-LTO.bc +; RUN: llvm-as %p/Inputs/main-LTO.ll -o %t/main-LTO.bc +; RUN: %gold -plugin %llvmshlibdir/LLVMgold%shlibext -o %t/foo-LTO %t/a-LTO.bc %t/main-LTO.bc +; RUN: obj2yaml %t/foo-fatLTO > %t/foo-fatLTO.yaml +; RUN: obj2yaml %t/foo-LTO > %t/foo-LTO.yaml +; RUN: diff %t/foo-fatLTO.yaml %t/foo-LTO.yaml + +;; Check if the no-LTO target executable produced from FatLTO object file is +;; identical to the one produced from regular object files + +; RUN: llc %p/Inputs/a-LTO.ll --filetype=obj -o %t/a.o +; RUN: llc %p/Inputs/main-LTO.ll --filetype=obj -o %t/main.o + +; RUN: %gold -o %t/foo-noLTO %t/a.o %t/main.o +; RUN: obj2yaml %t/foo-fatNoLTO > %t/foo-fatNoLTO.yaml +; RUN: obj2yaml %t/foo-noLTO > %t/foo-noLTO.yaml +; RUN: diff %t/foo-fatNoLTO.yaml %t/foo-noLTO.yaml diff --git a/llvm/tools/gold/gold-plugin.cpp b/llvm/tools/gold/gold-plugin.cpp --- a/llvm/tools/gold/gold-plugin.cpp +++ b/llvm/tools/gold/gold-plugin.cpp @@ -538,14 +538,6 @@ BufferRef = Buffer->getMemBufferRef(); } - // Only use bitcode files for LTO. InputFile::create() will load bitcode - // from the .llvmbc section within a binary object, this bitcode is typically - // generated by -fembed-bitcode and is not to be used by LLVMgold.so for LTO. - if (identify_magic(BufferRef.getBuffer()) != file_magic::bitcode) { - *claimed = 0; - return LDPS_OK; - } - *claimed = 1; Expected> ObjOrErr = InputFile::create(BufferRef);