[PATCH] D112127: [Codegen] Set ARITH_FENCE as meta-instruction

[LuoYuanke via Phabricator via llvm-commits]

LuoYuanke added inline comments.


================
Comment at: llvm/unittests/MIR/MachineMetadata.cpp:312
+  !8 = distinct !DISubprogram(name: "test0", scope: !1, file: !1, line: 1, type: !9, scopeLine: 1, flags: DIFlagPrototyped, spFlags: DISPFlagDefinition, unit: !0, retainedNodes: !12)
+  !9 = !DISubroutineType(types: !10)
+  !10 = !{null, !11}
----------------
Can we remove debug metadata to reduce the case?


================
Comment at: llvm/unittests/MIR/MachineMetadata.cpp:347
+    MachineInstr &MI = *It;
+    MI.dump();
+    ASSERT_TRUE(MI.isMetaInstruction());
----------------
Is dump() necessary?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112127