[PATCH] D69097: [AArch64][MachineOutliner] Return address signing for outlined functions

[Oliver Stannard (Linaro) via Phabricator via llvm-commits]

ostannard requested changes to this revision.
ostannard added inline comments.
This revision now requires changes to proceed.


================
Comment at: llvm/test/CodeGen/AArch64/machine-outliner-retaddr-sign-v8-3.ll:61
+; CHECK-NEXT:               .cfi_negate_ra_state
+; CHECK:                    bl OUTLINED_FUNC
+  %1 = alloca i32, align 4
----------------
I don't think this is correct: `@c` is being compiled for Armv8.0-A, but we've introduced a call to a function with the `retab` instruction, so it will fail if run on an v8.0-A machine. I think we need to check that all of the outlining candidate functions are being compiled for v8.3-A, or carefully construct the new function's subtarget to be a subset of the candidate functions.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69097