[llvm] [CodeGen] Fix InstructionCount remarks for MI bundles (PR #107621)

Francis Visoiu Mistrih via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 9 11:16:00 PDT 2024


francisvm wrote:

> > > This is probably by design. Code is supposed to logically treat a bundle as one instruction
> > 
> > 
> > Some targets use bundles to mark instruction packet boundaries, no? IMO, that shouldn't be counted as one instruction.
> 
> Yes, that was the original use (assuming some details about how packets are represented). I don't know what "InstructionCount" here is supposed to really mean though

I think regardless of bundles, they're still real MachineInstrs that get emitted, I don't see why they wouldn't get counted as regular instructions. I can see this count being a problem for pseudos that get expanded directly into MCInsts, but not for bundles.

https://github.com/llvm/llvm-project/pull/107621


More information about the llvm-commits mailing list