================ @@ -450,9 +450,9 @@ bool FlattenCFGOpt::MergeIfRegion(BasicBlock *BB, IRBuilder<> &Builder) { if (!CompareIfRegionBlock(IfTrue1, IfTrue2, SecondEntryBlock)) return false; } else if (IfTrue1 == FirstEntryBlock) { - // The then-path is empty, so we must use "and" operation to combine the + // The else-path is empty, so we must use "or" operation to combine the // conditions. - CombineOp = BinaryOperator::And; + CombineOp = BinaryOperator::Or; ---------------- arsenm wrote:
Was this really just completely backwards? https://github.com/llvm/llvm-project/pull/72522 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits