[PATCH] D42149: [GISel]: Refactor InstructionSelector::constrainSelectInstOperands

Aditya Nandakumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 16 16:48:30 PST 2018


aditya_nandakumar created this revision.
aditya_nandakumar added reviewers: dsanders, qcolombet, ab, aemerson.
Herald added subscribers: javed.absar, nhaehnle, arsenm.

This patch moves the constrainSelectedInstRegOperands from InstructionSelector into GlobalISel/Utils so it can be used elsewhere.
Also added constrainAllUses method into MachineInstrBuilder which forwards the call to utils. So now we can use it like,

BuildMI(...)

  .addUse(..)
  .addImm(..)
  .constrainAllUses(TII, TRI, RBI);


Repository:
  rL LLVM

https://reviews.llvm.org/D42149

Files:
  include/llvm/CodeGen/GlobalISel/InstructionSelector.h
  include/llvm/CodeGen/GlobalISel/InstructionSelectorImpl.h
  include/llvm/CodeGen/GlobalISel/Utils.h
  include/llvm/CodeGen/MachineInstrBuilder.h
  lib/CodeGen/GlobalISel/InstructionSelector.cpp
  lib/CodeGen/GlobalISel/Utils.cpp
  lib/Target/AArch64/AArch64InstructionSelector.cpp
  lib/Target/AMDGPU/AMDGPUInstructionSelector.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D42149.130067.patch
Type: text/x-patch
Size: 9773 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180117/b8ff1406/attachment.bin>


More information about the llvm-commits mailing list