CheerpWriter: changed checkDefined to checkTrue

The previous logic of checkDefined was causing issues when compiling
with the no-optimization flag together with -cheep-bounds-check.
Instead of checking if a value is undefined and throwing an error,
it is better to check if the member of a struct exists.

Author: Andres Madrid Ucha

llvm/include/llvm/Cheerp/Writer.h

@@ -750,11 +750,11 @@ class CheerpWriter final : public CheerpBaseWriter
 	/**
 	 * Compile a function to assure a GEP property access is defined
 	 */
-	void compileCheckDefined(const llvm::Value* p, bool needsOffset);
+	void compileCheckTrue(const llvm::Value* p, bool needsOffset);
 	/**
 	 * Compile a function for checking if a reference is defined
 	 */
-	void compileCheckDefinedHelper();
+	void compileCheckTrueHelper();
 	/**
 	 * Compile a JS string while escaping special characters
 	 */

llvm/lib/CheerpWriter/CheerpWriter.cpp

@@ -4550,7 +4550,7 @@ void CheerpWriter::compileLoad(const LoadInst& li, PARENT_PRIORITY parentPrio)
 					needsCheckBounds = true;
 					bool needsOffset = !li.use_empty() && Ty->isPointerTy() && PA.getPointerKindAssert(&li) == SPLIT_REGULAR && !PA.getConstantOffsetForPointer(&li);
 					stream<<"(";
-					compileCheckDefined(ptrOp, needsOffset);
+					compileCheckTrue(ptrOp, needsOffset);
 					stream<<",";
 			}
 	}
@@ -4765,7 +4765,7 @@ void CheerpWriter::compileStore(const StoreInst& si)
 		else if(ptrKind == COMPLETE_OBJECT && isGEP(ptrOp))
 		{
 			bool needsOffset = Ty->isPointerTy() && PA.getPointerKindAssert(&si) == SPLIT_REGULAR && !PA.getConstantOffsetForPointer(&si);
-			compileCheckDefined(ptrOp, needsOffset);
+			compileCheckTrue(ptrOp, needsOffset);
 			stream<<",";
 		}
 	}
@@ -6257,20 +6257,42 @@ void CheerpWriter::compileCheckBounds(const Value* p)
 	stream<<")";
 }
 
-void CheerpWriter::compileCheckDefinedHelper()
+void CheerpWriter::compileCheckTrueHelper()
 {
-	stream << "function checkDefined(m){if(m===undefined) throw new Error('UndefinedMemberAccess');}" << NewLine;
+	stream << "function checkTrue(m){if(!m) throw new Error('UndefinedMemberAccess');}" << NewLine;
 }
 
-void CheerpWriter::compileCheckDefined(const Value* p, bool needsOffset)
+void CheerpWriter::compileCheckTrue(const Value* p, bool needsOffset)
 {
-	// When compiling a SPIT_REGULAR, if there is an offset, we only check that.
-	// If the offset exists the base is guaranteed to exists in the type.
-	stream<<"checkDefined(";
-	compileGEP(cast<User>(p),COMPLETE_OBJECT, HIGHEST);
-	if(needsOffset)
-		stream << "o";
-	stream<<")";
+	const llvm::User* gep_inst = cast<User>(p);
+	SmallVector< const Value*, 8 > indices(std::next(gep_inst->op_begin()), gep_inst->op_end());
+	Type* basePointedType = cast<GEPOperator>(gep_inst)->getSourceElementType();
+
+	StructType* containerStructType = dyn_cast<StructType>(GetElementPtrInst::getIndexedType(basePointedType,
+		makeArrayRef(const_cast<Value* const*>(indices.begin()),
+					const_cast<Value* const*>(indices.end() - 1))));
+
+	if(containerStructType && indices.size() > 1)
+	{
+		assert(isa<ConstantInt>(indices.back()));
+	}
+
+	if (containerStructType) {
+		stream << "checkTrue('";
+		const Value* lastOperand = gep_inst->getOperand(gep_inst->getNumOperands() - 1);
+		const APInt& index = cast<Constant>(lastOperand)->getUniqueInteger();
+		uint64_t idxVal = index.getLimitedValue();
+
+		stream << types.getPrefixCharForMember(PA, containerStructType, idxVal) << idxVal;
+		if (needsOffset)
+			stream << "o";
+		stream << "' in ";
+		compileCompleteObject(gep_inst->getOperand(0) , indices.front());
+		if (indices.size() > 1) {
+			compileAccessToElement(basePointedType, makeArrayRef(std::next(indices.begin()), std::prev(indices.end())), false);
+		}
+		stream << ")";
+	}
 }
 
 void CheerpWriter::compileCheckBoundsAsmJSHelper()
@@ -6617,7 +6639,7 @@ void CheerpWriter::compileHelpers()
 	if ( checkBounds )
 	{
 		compileCheckBoundsHelper();
-		compileCheckDefinedHelper();
+		compileCheckTrueHelper();
 	}
 
 	compileBuiltins(false);