[clang] [aarch64] Add support for the __{inc|add}x18{byte|word|dword|qword intrinsics (PR #117752)
Daniel Paoliello via cfe-commits
cfe-commits at lists.llvm.org
Tue Dec 3 14:57:50 PST 2024
dpaoliello wrote:
> Sorry, just spotted an issue: I don't think you're emitting the operations in the correct order. You need to emit both arguments before you read the register or load the value.
>
> If you have some non-trivial operation like a call in one of the arguments, you don't want that call to happen in the middle of the operation. MSVC appears to emit the arguments before touching x18.
Done, and fixed the read/write x18 intrinsics as well.
https://github.com/llvm/llvm-project/pull/117752
More information about the cfe-commits
mailing list