[PATCH] D17356: Fix PR26655: Bail out if all the registers of an instruction in a BUNDLE already have the correct kill flag.

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 25 11:51:58 PDT 2016


t.p.northover added a comment.

Yep, sorry about the delay. That addition to addRegisterKilled needs some careful thought, since this isn't the only user.

The debug operand is probably fine, but it seems completely unobvious to me that all other users will want to skip internal reads, so we've got to think about all other call-sites. Not helped by the fact that I can't even see why it's necessary in **this** case yet.

I haven't forgotten though!

Tim.


Repository:
  rL LLVM

http://reviews.llvm.org/D17356





More information about the llvm-commits mailing list