[PATCH] D123334: [BOLT] Update skipRelocation for aarch64

[Rafael Auler via Phabricator via llvm-commits]

rafauler added inline comments.


================
Comment at: bolt/lib/Core/Relocation.cpp:227
 
+  // The ld might relax ADRP+ADD or ADRP+LDR sequences to the ADR+NOP
+  switch (Type) {
----------------
yota9 wrote:
> rafauler wrote:
> > By ld, do you mean GNU ld? How is the test testing for that? (since it uses lld)
> No, I meant generic linker, not bfd. The lld has this optimization too (https://reviews.llvm.org/D117614)
Can you replace ld by "linker", then?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123334/new/

https://reviews.llvm.org/D123334