This revision was not accepted when it landed; it landed in state "Needs
Revision".
This revision was automatically updated to reflect the committed changes.
Closed by commit rG880f39af6115: [Clang] Enable AIX initial-exec TLS mode
(authored by qiucf).
Repository:
rG LLVM Github Monorepo
CHAN
shchenz accepted this revision as: shchenz.
shchenz added a comment.
LGTM
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156076/new/
https://reviews.llvm.org/D156076
___
cfe-commits mailing list
cfe-commi
qiucf added a comment.
Ping
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D156076/new/
https://reviews.llvm.org/D156076
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-b
nemanjai added a comment.
So we don't have code generation for this, but we are enabling it in the front
end? What happens if we try to produce code for the IR this produces? It would
make more sense to me to implement what is needed in the back end prior to
allowing the front end to produce th
nemanjai requested changes to this revision.
nemanjai added a comment.
This revision now requires changes to proceed.
Please provide a description justifying this change. There is no context here
for the viewer to determine whether this change makes sense.
Repository:
rG LLVM Github Monorepo
qiucf created this revision.
qiucf added reviewers: stefanp, shchenz, amyk, tingwang, PowerPC.
Herald added a subscriber: nemanjai.
Herald added a reviewer: aaron.ballman.
Herald added a project: All.
qiucf requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: