Skip to content

Commit

Permalink
[ELF] - handle of command line options incompatible with -r
Browse files Browse the repository at this point in the history
Patch handles the command line options
incompatible with relocatable output.

Differential revision: http://reviews.llvm.org/D17758

llvm-svn: 262377
  • Loading branch information
George Rimar committed Mar 1, 2016
1 parent 7ea02fc commit 262b927
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 10 deletions.
9 changes: 8 additions & 1 deletion lld/ELF/Driver.cpp
Expand Up @@ -141,8 +141,15 @@ static void checkOptions(opt::InputArgList &Args) {
if (Config->EMachine == EM_AMDGPU && !Config->Entry.empty())
error("-e option is not valid for AMDGPU.");

if (Config->Relocatable && Config->Shared)
if (!Config->Relocatable)
return;

if (Config->Shared)
error("-r and -shared may not be used together");
if (Config->GcSections)
error("-r and --gc-sections may not be used together");
if (Config->ICF)
error("-r and --icf may not be used together");
}

static StringRef
Expand Down
17 changes: 17 additions & 0 deletions lld/test/ELF/driver.test
Expand Up @@ -20,6 +20,23 @@
# RUN: ld.lld --version 2>&1 | FileCheck -check-prefix=VERSION %s
# VERSION: LLD

## Attempt to link DSO with -r
# RUN: ld.lld -shared %t -o %t.so
# RUN: not ld.lld -r %t.so %t -o %tfail 2>&1 | FileCheck -check-prefix=ERR %s
# ERR: Attempted static link of dynamic object

## Attempt to use -r and -shared together
# RUN: not ld.lld -r -shared %t -o %tfail 2>&1 | FileCheck -check-prefix=ERR2 %s
# ERR2: -r and -shared may not be used together

## Attempt to use -r and --gc-sections together
# RUN: not ld.lld -r --gc-sections %t -o %tfail 2>&1 | FileCheck -check-prefix=ERR3 %s
# ERR3: -r and --gc-sections may not be used together

## Attempt to use -r and --icf together
# RUN: not ld.lld -r --icf=all %t -o %tfail 2>&1 | FileCheck -check-prefix=ERR4 %s
# ERR4: -r and --icf may not be used together

.globl _start
_start:
nop
9 changes: 0 additions & 9 deletions lld/test/ELF/relocatable.s
Expand Up @@ -15,15 +15,6 @@
# RUN: ld.lld -e main %t -o %texec
# RUN: llvm-readobj -file-headers %texec | FileCheck -check-prefix=CHECKEXE %s

## Attempt to link DSO with -r
# RUN: ld.lld -shared %t1.o -o %t.so
# RUN: not ld.lld -r %t.so %t2.o -o %tfail 2>&1 | FileCheck -check-prefix=ERR %s
# ERR: Attempted static link of dynamic object

## Attempt to use -r and shared together
# RUN: not ld.lld -r -shared %t2.o -o %tfail 2>&1 | FileCheck -check-prefix=ERR2 %s
# ERR2: -r and -shared may not be used together

# CHECK: ElfHeader {
# CHECK-NEXT: Ident {
# CHECK-NEXT: Magic: (7F 45 4C 46)
Expand Down

0 comments on commit 262b927

Please sign in to comment.