[PATCH] D116945: [MachineInstr] Don't include debug uses in bundle header (PR52817)

[Nikita Popov via Phabricator via llvm-commits]

nikic created this revision.
nikic added a reviewer: MatzeB.
Herald added subscribers: kerbowa, dmgreen, hiraditya, nhaehnle, jvesely.
nikic requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Following https://github.com/llvm/llvm-project/issues/52817#issuecomment-1007635426, this excludes debug instructions when finalizing the bundle. Uses in debug instructions will no longer be included in the BUNDLE header.

This also ends up dropping the "internal" modifier for debug instructions -- not sure whether that's good or bad. If that should be retained, I can change the patch accordingly.

Fixes https://github.com/llvm/llvm-project/issues/52817.


https://reviews.llvm.org/D116945

Files:
  llvm/lib/CodeGen/MachineInstrBundle.cpp
  llvm/test/CodeGen/AMDGPU/postra-bundle-memops.mir
  llvm/test/CodeGen/Thumb2/mve-vpt-block-debug.mir
  llvm/test/CodeGen/Thumb2/pr52817.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D116945.398632.patch
Type: text/x-patch
Size: 10036 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220110/323e9ece/attachment.bin>