Skip overlapping AABB checks if the block is a cube

pmmp · Jan 27, 2025 · cd8a37d · cd8a37d
1 parent 4c7a0df
commit cd8a37d
Showing 1 changed file with 19 additions and 15 deletions.
diff --git a/src/world/World.php b/src/world/World.php
@@ -1575,26 +1575,30 @@ private function getBlockCollisionBoxesForCell(int $x, int $y, int $z, array $co
 			?->getBlockStateId($x & Chunk::COORD_MASK, $y, $z & Chunk::COORD_MASK) ?? Block::EMPTY_STATE_ID;
 
 		$cellBB = null;
-		$boxes = match($collisionInfo[$stateId]){
+		$stateCollisionInfo = $collisionInfo[$stateId] ?? throw new AssumptionFailedError("This should always exist");
+		$boxes = match($stateCollisionInfo){
 			RuntimeBlockStateRegistry::COLLISION_NONE => [],
 			RuntimeBlockStateRegistry::COLLISION_CUBE => [$cellBB = AxisAlignedBB::one()->offset($x, $y, $z)],
 			default => $this->getBlockAt($x, $y, $z)->getCollisionBoxes()
 		};
 
-		foreach(Facing::OFFSET as [$dx, $dy, $dz]){
-			$offsetY = $y + $dy;
-			if($offsetY < $this->minY || $offsetY > $this->maxY){
-				continue;
-			}
-			$stateId = $this
-				->getChunk(($x + $dx) >> Chunk::COORD_BIT_SIZE, ($z + $dz) >> Chunk::COORD_BIT_SIZE)
-				?->getBlockStateId(($x + $dx) & Chunk::COORD_MASK, $offsetY, ($z + $dz) & Chunk::COORD_MASK) ?? Block::EMPTY_STATE_ID;
-			if($collisionInfo[$stateId] === RuntimeBlockStateRegistry::COLLISION_MAY_OVERFLOW){
-				$cellBB ??= AxisAlignedBB::one()->offset($x, $y, $z);
-				$extraBoxes = $this->getBlockAt($x + $dx, $offsetY, $z + $dz)->getCollisionBoxes();
-				foreach($extraBoxes as $extraBox){
-					if($extraBox->intersectsWith($cellBB)){
-						$boxes[] = $extraBox;
+		//overlapping AABBs can't make any difference if this is a cube, so we can save some CPU cycles in this common case
+		if($stateCollisionInfo !== RuntimeBlockStateRegistry::COLLISION_CUBE){
+			foreach(Facing::OFFSET as [$dx, $dy, $dz]){
+				$offsetY = $y + $dy;
+				if($offsetY < $this->minY || $offsetY > $this->maxY){
+					continue;
+				}
+				$stateId = $this
+					->getChunk(($x + $dx) >> Chunk::COORD_BIT_SIZE, ($z + $dz) >> Chunk::COORD_BIT_SIZE)
+					?->getBlockStateId(($x + $dx) & Chunk::COORD_MASK, $offsetY, ($z + $dz) & Chunk::COORD_MASK) ?? Block::EMPTY_STATE_ID;
+				if($collisionInfo[$stateId] === RuntimeBlockStateRegistry::COLLISION_MAY_OVERFLOW){
+					$cellBB ??= AxisAlignedBB::one()->offset($x, $y, $z);
+					$extraBoxes = $this->getBlockAt($x + $dx, $offsetY, $z + $dz)->getCollisionBoxes();
+					foreach($extraBoxes as $extraBox){
+						if($extraBox->intersectsWith($cellBB)){
+							$boxes[] = $extraBox;
+						}
 					}
 				}
 			}