Also I think the sense of this comment is backwards:

// Or, if we are allowed to create dynamic relocations against
// read-only sections (i.e. unless "-z notext" is given),
// we can create a dynamic relocation as we want, too.
if (!Config->ZText)
return Expr;

That should be "i.e., if "-z notext" is given"

Also I think the sense of this comment is backwards:

// Or, if we are allowed to create dynamic relocations against
// read-only sections (i.e. unless "-z notext" is given),
// we can create a dynamic relocation as we want, too.
if (!Config->ZText)
return Expr;

That should be "i.e., if "-z notext" is given"

Looking at this.

But you also helped to find bug in google translate :)
It translates "unless" to russian "if", and not to "if ... not":
https://translate.google.ru/#en/ru/unless
notice, when pressing switch message button, backward translation from
russian is "if" then.

Reduced test case is following.

main.s:
.text
_start:
callq atexit

dso.s:
.text
.global atexit
atexit:
nop

llvm-mc -filetype=obj -triple=x86_64-pc-linux main.s -o main.o
llvm-mc -filetype=obj -triple=x86_64-pc-linux dso.s -o dso.o
ld.lld -shared dso.o -o dso.so

ld.lld test.o -o out dso.so -z notext

error: relocation R_X86_64_PC32 cannot be used against shared object; recompile with -fPIC

Fix: https://reviews.llvm.org/D41541

First fix for 'hello world' was committed as r321400.

Second fix is https://reviews.llvm.org/D41551,
I checked that 'hello world' provided links fine with it.

Last piece: r321473.