Fix operator precedence in StackLimitEvader memory guard assertion

Add parentheses so the reservedMemory and requiredSlots bounds correctly evaluate
to 2^32. Without them, 'u256(1) << 32 - 1' is parsed as 'u256(1) << (32 - 1)' = 2^31
due to C++ operator precedence. Use '(u256(1) << 32)' and '(uint64_t(1) << 32)' for
clearer semantics that allow the full 32-bit range [0, 2^32-1].

Author: cuiweixie

libyul/optimiser/StackLimitEvader.cpp

@@ -210,7 +210,7 @@ void StackLimitEvader::run(
 
 	// Make sure all calls to ``memoryguard`` we found have the same value as argument (otherwise, abort).
 	u256 reservedMemory = literalArgumentValue(*memoryGuardCalls.front());
-	yulAssert(reservedMemory < u256(1) << 32 - 1, "");
+	yulAssert(reservedMemory < (u256(1) << 32), "");
 
 	for (FunctionCall const* memoryGuardCall: memoryGuardCalls)
 		if (reservedMemory != literalArgumentValue(*memoryGuardCall))
@@ -230,7 +230,7 @@ void StackLimitEvader::run(
 
 	MemoryOffsetAllocator memoryOffsetAllocator{_unreachableVariables, callGraph.functionCalls, functionDefinitions};
 	uint64_t requiredSlots = memoryOffsetAllocator.run();
-	yulAssert(requiredSlots < (uint64_t(1) << 32) - 1, "");
+	yulAssert(requiredSlots < (uint64_t(1) << 32), "");
 
 	StackToMemoryMover::run(_context, reservedMemory, memoryOffsetAllocator.slotAllocations, requiredSlots, _astRoot);