[PATCH] D130676: [AArch64] Treat x18 as callee-saved in functions with Windows calling convention on Darwin

[Martin Storsjö via Phabricator via llvm-commits]

mstorsjo added a comment.

I think this looks reasonable, but I’ll have a closer read later.

Where did you trigger this issue - by building Wine on macOS? (I did try building Wine on macOS/arm64 in 2020, and the logic for backing up x18 landed in 2019, so I wonder why I didn’t notice this issue… Maybe the Xcode clang didn’t contain that change yet at that point, or maybe it’s not noticed or the compiler is built without assertions?)

FWIW, with a recent refactoring in Wine, where a proper syscall interface is used between windows code and the native unix code, I think the whole mechanism for backing up/restoring x18 might be unnecessary, so we could consider ripping it out altogether too. I guess I could run some tests to see what the current state is…


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D130676