[LLVMdev] Problems with DAG Combiner

Jakob Stoklund Olesen stoklund at 2pi.dk
Sun Aug 23 10:27:25 PDT 2009


On 23/08/2009, at 19.12, Jakob Stoklund Olesen wrote:

> When LegalOperations is set, the DAG combiner must not create illegal
> operations. It is a bug if it does. I recently fixed this in the first
> if statement in SimplifyBinOpWithSameOpcodeHands(). Perhaps you could
> add a check to the second if statement and submit a patch?

Sorry, the second 'if' is fine, I think. Do you have the patch from  
r78497? Are you still getting illegal ORs?




More information about the llvm-dev mailing list