This is an archive of the discontinued LLVM Phabricator instance.

Correctly report modified status for ObjCARCContract
ClosedPublic

Authored by serge-sans-paille on Jun 5 2020, 12:20 AM.

Reviewers

Commits

Summary

Seems obviously correct.

This revision is now accepted and ready to land.Jun 5 2020, 5:57 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

llvm/

lib/

Transforms/

ObjCARC/

1 line

Show First 20 Lines • Show All 523 Lines • ▼ Show 20 Lines	case ARCInstKind::User:
// Be conservative if the function has any alloca instructions.		// Be conservative if the function has any alloca instructions.
// Technically we only care about escaping alloca instructions,		// Technically we only care about escaping alloca instructions,
// but this is sufficient to handle some interesting cases.		// but this is sufficient to handle some interesting cases.
if (isa<AllocaInst>(Inst))		if (isa<AllocaInst>(Inst))
TailOkForStoreStrongs = false;		TailOkForStoreStrongs = false;
return true;		return true;
case ARCInstKind::IntrinsicUser:		case ARCInstKind::IntrinsicUser:
// Remove calls to @llvm.objc.clang.arc.use(...).		// Remove calls to @llvm.objc.clang.arc.use(...).
		Changed = true;
Inst->eraseFromParent();		Inst->eraseFromParent();
return true;		return true;
default:		default:
return true;		return true;
}		}
}		}

//===----------------------------------------------------------------------===//		//===----------------------------------------------------------------------===//
▲ Show 20 Lines • Show All 211 Lines • Show Last 20 Lines