[PATCH] Fixup PHI nodes in LowerSwitch
Pete
peter_cooper at apple.com
Wed Jul 9 10:55:10 PDT 2014
>>! In D4298#25, @kariddi wrote:
> Reverted the to the getFirstNonPHI() version waiting for more input on the matter
Yeah, from what David said I was wrong about this so good to proceed with the original PHI looping code.
I did take a look in Verifier::visitBasicBlock(BasicBlock &BB) and we don't actually check that PHIs are the first instructions in a BB, or contiguous, but that should be easy to add as David points out its what LangRef says already.
http://reviews.llvm.org/D4298
More information about the llvm-commits
mailing list