[PATCH] D17659: [X86] Move the ATOMIC_LOAD_OP ISel from DAGToDAG to ISelLowering. NFCI.

Ahmed Bougacha via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 26 14:46:40 PST 2016


ab created this revision.
ab added reviewers: jfb, t.p.northover.
ab added subscribers: llvm-commits, qcolombet.

This is long-standing dirtiness, as acknowledged by r77582:

    The current trick is to select it into a merge_values with
    the first definition being an implicit_def. The proper solution is
    to add new ISD opcodes for the no-output variant.

Doing this before selection will let us combine away some constructs.

http://reviews.llvm.org/D17659

Files:
  lib/Target/X86/X86ISelDAGToDAG.cpp
  lib/Target/X86/X86ISelLowering.cpp
  lib/Target/X86/X86ISelLowering.h
  lib/Target/X86/X86InstrCompiler.td
  lib/Target/X86/X86InstrInfo.td

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D17659.49250.patch
Type: text/x-patch
Size: 26831 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160226/1fd6811a/attachment.bin>


More information about the llvm-commits mailing list