[PATCH] D24073: [AArch64] Support stackmap/patchpoint in getInstSizeInBytes

[Diana Picus via llvm-commits]

rovka added a comment.

In https://reviews.llvm.org/D24073#539438, @kavon wrote:

> When the op code is a patchpoint, isn't a call still emitted? I think maybe the total number of bytes returned for patchpoints should include the bytes for the call.


>From my reading of AArch64AsmPrinter::LowerPATCHPOINT, it looks like the number of bytes requested by the patchpoint has to include the bytes for the call - i.e. if the call is generated, its size is subtracted from the number of bytes requested, and only the remaining number of nops are added.


https://reviews.llvm.org/D24073