<html><head><meta http-equiv="Content-Type" content="text/html charset=iso-8859-1"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>Hi, Daniel. Sorry for not replying sooner -- I was on vacation.</div><div><br></div><div>I agree that this should be changed, but I'm not sure the new phrasing is strong enough. We're not actually checking what the behavior of a 0-byte allocation is on the user's target platform, so it's possible the use is unsafe <i>now,</i> not just when they try to port it to another target. How about "allocation of 0 bytes has unspecified behavior"? It's a bit more wordy, but it also sounds more like something to pay attention to.</div><div><br></div><div>(I thought about "implementation-defined behavior" as well, but I'm not sure that's strong enough either...people might just assume they know how their own implementation works. Then again, it <i>is</i> documented for many platforms.)</div><div><br></div><div>What do you think?</div><div>Jordan</div><div><br></div><br><div><div>On Apr 20, 2014, at 4:46 , Daniel Marjamäki <<a href="mailto:Daniel.Marjamaki@evidente.se">Daniel.Marjamaki@evidente.se</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><br>Hello!<br><br>Here is a fix for:<br><a href="http://llvm.org/bugs/show_bug.cgi?id=15708">http://llvm.org/bugs/show_bug.cgi?id=15708</a><br><br>The message 'Undefined allocation of 0 bytes' is not entirely correct. The behaviour is implementation defined.<br><br>Best regards,<br>Daniel Marjamäki<br><br>..................................................................................................................<br>Daniel Marjamäki Senior Engineer<br>Evidente ES East AB Warfvinges väg 34 SE-112 51 Stockholm Sweden<br><br>Mobile: +46 (0)709 12 42 62<br>E-mail: Daniel.Marjamaki@evidente.se<br><br>www.evidente.se<br><span><15708.diff></span>_______________________________________________<br>cfe-commits mailing list<br>cfe-commits@cs.uiuc.edu<br>http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits<br></blockquote></div><br></body></html>