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

[Renato Golin via llvm-commits]

rengolin added inline comments.

================
Comment at: lib/Target/AArch64/AArch64InstrInfo.cpp:64
@@ -61,1 +63,3 @@
+  //        before the assembly printer.
+  unsigned NumBytes = 0;
   const MCInstrDesc &Desc = MI.getDesc();
----------------
Nit: maybe make the switch set NumBytes and return it in the end?

The unreachable down there is silly, but probably silences a compiler warning. This way, we could get rid of that, too.


https://reviews.llvm.org/D24073