[all-commits] [llvm/llvm-project] 8e8e31: [ARM] Fix in ICE when retrieving the number of mic...

Momchil Velikov via All-commits all-commits at lists.llvm.org
Fri Dec 13 10:20:00 PST 2019


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 8e8e3181aa52301f505a7445f05b21d7fc6882a9
      https://github.com/llvm/llvm-project/commit/8e8e3181aa52301f505a7445f05b21d7fc6882a9
  Author: Momchil Velikov <momchil.velikov at arm.com>
  Date:   2019-12-13 (Fri, 13 Dec 2019)

  Changed paths:
    M llvm/lib/Target/ARM/ARMInstrVFP.td
    A llvm/test/CodeGen/ARM/vlldm-vlstm-uops.mir

  Log Message:
  -----------
  [ARM] Fix in ICE when retrieving the number of micro-ops for vlldm/vlstm

The big switch in `ARMBaseInstrInfo::getNumMicroOps` is missing cases for
`VLLDM` and `VLSTM`, which are currently defined with itineraries having a
dynamic count of micro-ops.

Assuming an optimistic case in which these instruction do not actually perform
loads or stores, and with the idea that Armv8-m cores are supposed to use the
new style scheduling models, this patch just sets the itinerary for those two
instructions to `NoItinerary`.

Differential Revision: https://reviews.llvm.org/D71266




More information about the All-commits mailing list