[PATCH] D22372: [CodeGen] Refactor MachineMemOperand::Flags's target-specific flags.

Justin Lebar via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 14 11:08:22 PDT 2016


jlebar created this revision.
jlebar added a reviewer: MatzeB.
jlebar added a subscriber: llvm-commits.

Make the target-specific flags in MachineMemOperand::Flags real, bona
fide enum values.  This simplifies users, prevents various constants
from going out of sync, and avoids the false sense of security provided
by declaring static members in classes and then forgetting to define
them inside of cpp files.

https://reviews.llvm.org/D22372

Files:
  include/llvm/CodeGen/MachineMemOperand.h
  lib/CodeGen/MachineInstr.cpp
  lib/Target/AArch64/AArch64InstrInfo.cpp
  lib/Target/AArch64/AArch64InstrInfo.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D22372.64011.patch
Type: text/x-patch
Size: 4490 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160714/1364536c/attachment.bin>


More information about the llvm-commits mailing list