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

[Renato Golin via llvm-commits]

rengolin added a comment.

Now that looks good to me.

I don't like much the tests, because if the branch relaxation algorithm change, this test can "fail" without really failing, and whoever hits this will have a hard time understanding how to make this test proper. But I have no better idea.

I'll leave this be for a while, see if anyone else has a better idea. If not, having those tests are better than nothing, and we can work on trying some unit tests in the future.


https://reviews.llvm.org/D24073