[PATCH] D23379: BranchRelaxation: Fix handling of blocks with multiple conditional branches

Tim Northover via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 16 10:11:13 PDT 2016

Previous message: [llvm] r266162 - Recommit r265547, and r265610, r265639, r265657 on top of it, plus
Next message: [llvm] r278822 - When the inline spiller rematerializes an instruction, take the debug location from the instruction
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

t.p.northover accepted this revision.
t.p.northover added a comment.
This revision is now accepted and ready to land.

This is pretty scary, I didn't know we blocks with 3 terminators. Looks like a reasonable fix for the uncommon case though.


https://reviews.llvm.org/D23379

Previous message: [llvm] r266162 - Recommit r265547, and r265610, r265639, r265657 on top of it, plus
Next message: [llvm] r278822 - When the inline spiller rematerializes an instruction, take the debug location from the instruction
Messages sorted by: [ date ] [ thread ] [ subject ] [ author ]

More information about the llvm-commits mailing list