[PATCH] D149378: [Docs][llvm-exegesis] Specify supported platforms and architectures

Aiden Grossman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Apr 29 00:35:39 PDT 2023


aidengrossman marked an inline comment as done.
aidengrossman added a comment.

Right. Some of it will work but a lot of it won't work and will probably crash. All of the parts that I'm aware of call into the `ExegesisTarget` implementations to get specific registers to use. In the case of `getScratchMemoryRegister`, for the x86 implementation, it does seem like Windows is supported as the function accepts a triple and changes the register depending upon the platform, so it seems like the ABI differences are better accounted for than I thought, and the the other snippet repetitor implementations, specifically the loop one, seem to take this into account when choosing registers.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149378



More information about the llvm-commits mailing list