<html><head></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><div>Duncan,</div><div><br></div>You are right. My mistake also got caught by test/CodeGen/X86/x86-64-malloc.ll when MallocInst is avoided (not the default mode).<div><br></div><div>Instead of switching away from ConstantExpr::getSizeOf, I fixed <span class="Apple-style-span" style="font-family: Menlo; font-size: 11px; ">isArrayMalloc() <span class="Apple-style-span" style="font-family: Helvetica; font-size: medium; ">to cast ElementSize to the right size.</span></span></div><div><font class="Apple-style-span" face="Menlo" size="3"><span class="Apple-style-span" style="font-size: 11px; "><br></span></font></div><div><font class="Apple-style-span" face="Menlo" size="3"><span class="Apple-style-span" style="font-size: 11px;"><span class="Apple-style-span" style="font-family: Helvetica; font-size: medium; ">Here is the new patch:</span></span></font></div><div></div></body></html>