[PATCH] D21448: Codegen: LICM Remove check for exactly 1 register def.

Kyle Butt via llvm-commits llvm-commits at lists.llvm.org
Tue Jun 21 16:26:33 PDT 2016


iteratee added a comment.

Quentin, Andrew,

Functionally this is a one-line change. When splitting an instruction from a memory op to an explicit load and a register op, we were checking the number of register defs in the resulting register op, and bailing if it wasn't exactly one. I've thought carefully about it, and the check doesn't seem to be necessary, and the tests all pass, but I'd like someone more familiar with that code to look over it.

Thanks,
Kyle.


http://reviews.llvm.org/D21448





More information about the llvm-commits mailing list