[PATCH] D142218: [MC] Store implicit ops immediately after the TargetInsts table. NFC.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/include/llvm/MC/MCInstrDesc.h:581
   ArrayRef<MCPhysReg> implicit_defs() const {
-    return {ImplicitOps + NumImplicitUses, NumImplicitDefs};
+    auto ImplicitOps = reinterpret_cast<const MCPhysReg *>(this + Opcode + 1);
+    return {ImplicitOps + ImplicitOffset + NumImplicitUses, NumImplicitDefs};
----------------
This looks a bit undefined to me. Is ubsan and asan happy with this?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142218