[PATCH] D136525: [M68k] Add codegen pattern for atomic load / store

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Oct 29 05:00:30 PDT 2022


RKSimon added a comment.

Please can you pre-commit pipeline.ll for current trunk and then rebase so that the patch shows the diff?



================
Comment at: llvm/lib/Target/M68k/M68kInstrAtomics.td:10
+// FIXME: This is only supported on MC68020 and later.
+class MxCASOp<bits<2> size_encoding, MxType type>
+    : MxInst<(outs type.ROp:$out),
----------------
Wrap this inside FeatureISA20 ?


================
Comment at: llvm/test/CodeGen/M68k/pipeline.ll:2
+; RUN: llc -mtriple=m68k -debug-pass=Structure < %s -o /dev/null 2>&1 | grep -v "Verify generated machine code" | FileCheck %s
+;CHECK:  ModulePass Manager
+; CHECK-NEXT:    Pre-ISel Intrinsic Lowering
----------------
; CHECK


================
Comment at: llvm/test/CodeGen/M68k/pipeline.ll:105
+; CHECK-NEXT:      Free MachineFunction
\ No newline at end of file

----------------
newline


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

https://reviews.llvm.org/D136525



More information about the llvm-commits mailing list