# HG changeset patch # User Frits van Bommel # Date 1235760085 -3600 # Node ID 45ed9e125a0095f41be257f07fd2d4f51d696f3e # Parent de97188378bcb95b1700fb5741bea150f6acd857 Remove initialization of padding of reals, ireals and creals. According to the spec and C ABIs, the contents of padding is undefined. This was breaking dstress creal_01,02,03,07,08,09 and 20 on x86-64. The code was apparently added to fix creal_13,14 and 15, which explicitly compare padding and assert if it's different. I'd argue those tests are broken. (Also, on x86-64 they *also* fail) The tests this fixes, on the other hand, are basic arithmetic. diff -r de97188378bc -r 45ed9e125a00 gen/llvmhelpers.cpp --- a/gen/llvmhelpers.cpp Fri Feb 27 12:35:31 2009 +0100 +++ b/gen/llvmhelpers.cpp Fri Feb 27 19:41:25 2009 +0100 @@ -1378,30 +1378,7 @@ { Logger::println("expression initializer"); assert(ex->exp); - DValue* res = ex->exp->toElem(gIR); - - assert(llvm::isa(target->getType()) && "init target must be ptr"); - const LLType* targetty = target->getType()->getContainedType(0); - if(targetty == LLType::X86_FP80Ty) - { - Logger::println("setting fp80 padding to zero"); - - LLValue* castv = DtoBitCast(target, getPtrToType(LLType::Int16Ty)); - LLValue* padding = DtoGEPi1(castv, 5); - DtoStore(llvm::Constant::getNullValue(LLType::Int16Ty), padding); - } - else if(targetty == DtoComplexType(Type::tcomplex80)) - { - Logger::println("setting complex fp80 padding to zero"); - - LLValue* castv = DtoBitCast(target, getPtrToType(LLType::Int16Ty)); - LLValue* padding = DtoGEPi1(castv, 5); - DtoStore(llvm::Constant::getNullValue(LLType::Int16Ty), padding); - padding = DtoGEPi1(castv, 11); - DtoStore(llvm::Constant::getNullValue(LLType::Int16Ty), padding); - } - - return res; + return ex->exp->toElem(gIR); } else if (init->isVoidInitializer()) {