InstrItineraryData is already on the subtarget, no reason to
cache it on the target as well.
Remove unnecessary caching of the subtarget for HexagonFrameLowering and remove the unused constructor argument.
Reduce indentation.
Make HexagonISelLowering not dependent upon a HexagonTargetMachine,
but a normal TargetMachine and remove a few cached uses.