[PATCH] D16826: Unify the target opcode enum in TargetOpcodes.h and the FixedInstrs array in CodeGenTarget.cpp to avoid the ordering dependence

David Kreitzer via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 2 14:42:19 PST 2016


DavidKreitzer created this revision.
DavidKreitzer added a reviewer: qcolombet.
DavidKreitzer added a subscriber: llvm-commits.
Herald added subscribers: sanjoy, aemerson.

Quentin, is this what you had in mind with your FIXME's from r258333?

The one effect of this change worth noting is that PRE_ISEL_GENERIC_OPCODE_END no longer gets its own value in the TargetOpcode enum. I think that is harmless and probably desirable.


http://reviews.llvm.org/D16826

Files:
  include/llvm/Target/TargetOpcodes.def
  include/llvm/Target/TargetOpcodes.h
  utils/TableGen/CodeGenTarget.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D16826.46700.patch
Type: text/x-patch
Size: 15211 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160202/a385e93b/attachment-0001.bin>


More information about the llvm-commits mailing list