`should_scavenge` first load the object from the passed-in arg (location) then checks if the object lives in young-gen. Since the loaded object is (often) needed by callers, for scavenging (copying to to-space/old-gen), the caller needs to re-load from the same location, which is unnecessary work.

Remove `should_scavenge` so that existing callers perform these two steps (loading and checking if-in-young-gen) manually to avoid the redundant work.