[PATCH] [AArch64] fix PR22408: clang no longer bootstraps itself on AArch64 linux

Kristof Beyls kristof.beyls at arm.com
Thu Feb 19 02:53:55 PST 2015


I've cleaned up one comment, and removed 3 FIXME's that after checking were not necessary:

- For local exec, we obviously don't need to produce fixed sequences as the linker won't relax that any further.
- For initial exec, no fixed sequence is necessary, as linkers don't need to and won't (checked in ld.bfd and ld.gold) assume a fixed sequence of instructions when relaxing from initial exec to local exec.
- The FIXME removed from the regression test should have been removed already in the initial patch: I had fixed up that test already.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D7677

Files:
  lib/Target/AArch64/AArch64AsmPrinter.cpp
  lib/Target/AArch64/AArch64CleanupLocalDynamicTLSPass.cpp
  lib/Target/AArch64/AArch64ISelLowering.cpp
  lib/Target/AArch64/AArch64ISelLowering.h
  lib/Target/AArch64/AArch64InstrInfo.td
  lib/Target/AArch64/AArch64MCInstLower.cpp
  lib/Target/AArch64/Utils/AArch64BaseInfo.h
  test/CodeGen/AArch64/arm64-tls-dynamics.ll
  test/CodeGen/AArch64/arm64-tls-execs.ll

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D7677.20267.patch
Type: text/x-patch
Size: 34214 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150219/cef708ce/attachment.bin>


More information about the llvm-commits mailing list