[PATCH] D71210: [MC] Widen the functional unit type from 32 to 64 bits.

[Bevin Hansson via Phabricator via llvm-commits]

ebevhan created this revision.
Herald added subscribers: llvm-commits, jsji, hiraditya.
Herald added a project: LLVM.

The type used to represent functional units in MC is
'unsigned', which is 32 bits wide. This is currently
not a problem in any upstream target as no one seems
to have hit the limit on this yet, but in our
downstream one, we need to define more than 32
functional units.

Increasing the size does not seem to cause a huge
size increase in the binary (an llc debug build went
from 1366497672 to 1366523984, a difference of 26k),
so perhaps it would be acceptable to have this patch
applied upstream as well.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D71210

Files:
  llvm/include/llvm/CodeGen/ScoreboardHazardRecognizer.h
  llvm/include/llvm/MC/MCInstrItineraries.h
  llvm/lib/CodeGen/MachinePipeliner.cpp
  llvm/lib/CodeGen/ScoreboardHazardRecognizer.cpp
  llvm/lib/Target/Hexagon/HexagonInstrInfo.cpp
  llvm/lib/Target/Hexagon/HexagonInstrInfo.h
  llvm/lib/Target/Hexagon/HexagonVLIWPacketizer.cpp
  llvm/utils/TableGen/SubtargetEmitter.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D71210.232852.patch
Type: text/x-patch
Size: 7708 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20191209/0630577a/attachment.bin>