[PATCH] Separate out MVT in a separate header file: MachineValueType.h

Patrik Hägglund H patrik.h.hagglund at ericsson.com
Wed Mar 12 05:59:05 PDT 2014


Here is a patch that splits out MVT from ValueTypes.h into a new heder file, MachineValueType.h. The idea is to make it easier to
ensure that stuff after type legalization only use MVT (rather than EVT), by watching include dependencies. The second patch utilizes this for all current trivial cases (that don't need any further restructuring).

I will also probably commit further cleanups regarding this.

Any comments regarding naming or further cleanups?

/Patrik Hägglund

Subject: [PATCH 1/2] Separate out MVT in a separate header file:
MachineValueType.h

The idea behind this split of ValueTypes.h, is to make it easier to
ensure that stuff after type legalization only use MVT (rather than
EVT), by watching include dependencies.
---
include/llvm/CodeGen/MachineValueType.h | 578 ++++++++++++++++++++++++++++++++
include/llvm/CodeGen/ValueTypes.h       | 561 +------------------------------
2 files changed, 581 insertions(+), 558 deletions(-)
create mode 100644 include/llvm/CodeGen/MachineValueType.h



Subject: [PATCH 2/2] Replace ValueTypes.h with MachineValueType.h if possible.

---
lib/Target/Hexagon/HexagonInstrInfo.h          | 2 ++
lib/Target/X86/InstPrinter/X86InstComments.cpp | 2 +-
lib/Target/X86/Utils/X86ShuffleDecode.cpp      | 2 +-
lib/Target/X86/X86RegisterInfo.cpp             | 2 +-
utils/TableGen/CodeGenInstruction.h            | 2 +-
utils/TableGen/CodeGenIntrinsics.h             | 2 +-
utils/TableGen/CodeGenRegisters.h              | 2 +-
utils/TableGen/DAGISelMatcher.h                | 2 +-
8 files changed, 9 insertions(+), 7 deletions(-)

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140312/3e0bee1f/attachment.html>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Separate-out-MVT-in-a-separate-header-file-MachineVa.patch
Type: application/octet-stream
Size: 39532 bytes
Desc: 0001-Separate-out-MVT-in-a-separate-header-file-MachineVa.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140312/3e0bee1f/attachment.obj>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0002-Replace-ValueTypes.h-with-MachineValueType.h-if-poss.patch
Type: application/octet-stream
Size: 4511 bytes
Desc: 0002-Replace-ValueTypes.h-with-MachineValueType.h-if-poss.patch
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140312/3e0bee1f/attachment-0001.obj>


More information about the llvm-commits mailing list