[all-commits] [llvm/llvm-project] 5a3a9e: [ELF][AArch64] Rename --force-bti to -z force-bti ...

Fangrui Song via All-commits all-commits at lists.llvm.org
Wed Dec 11 09:26:38 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 5a3a9e9927b714e94e1c1b839e17429806272052
      https://github.com/llvm/llvm-project/commit/5a3a9e9927b714e94e1c1b839e17429806272052
  Author: Fangrui Song <maskray at google.com>
  Date:   2019-12-11 (Wed, 11 Dec 2019)

  Changed paths:
    M lld/ELF/Driver.cpp
    M lld/ELF/Options.td
    M lld/docs/ld.lld.1
    M lld/test/ELF/aarch64-bti-pac-cli-error.s
    M lld/test/ELF/aarch64-feature-bti.s
    M lld/test/ELF/aarch64-feature-btipac.s
    M lld/test/ELF/aarch64-feature-pac.s

  Log Message:
  -----------
  [ELF][AArch64] Rename --force-bti to -z force-bti and --pac-plt to -z pac-plt

Summary:
The original design used --foo but the upstream complained that ELF only
options should be -z foo. See https://sourceware.org/ml/binutils/2019-04/msg00151.html
https://sourceware.org/git/?p=binutils-gdb.git;a=commitdiff;h=8bf6d176b0a442a8091d338d4af971591d19922c
made the rename.

Our --force-bti and --pac-plt implement the same functionality, so it
seems wise to be consistent with GNU ld.

Reviewed By: peter.smith

Subscribers: emaste, arichardson, kristof.beyls, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D71327




More information about the All-commits mailing list