[PATCH] D85456: Fix instruction counting in post-RA scheduler

Thu Aug 6 14:29:17 PDT 2020

rampitec added a comment.

In D85456#2201140 <https://reviews.llvm.org/D85456#2201140>, @arsenm wrote:

> In D85456#2201120 <https://reviews.llvm.org/D85456#2201120>, @kerbowa wrote:
>
>> Seems wrong to have a bundle without BUNDLE opcode but the change seems fine obviously. Since it's allowed we should probably just account for the possibility.
>
> But is it? I think this is a failing of the verifier and it is not allowed

It probably isn't. But there seems to be quite a few places which do not work with properly formed bundles. One of them is ExpandPostRA.

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D85456/new/

https://reviews.llvm.org/D85456