<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix">On 06/12/2014 12:28 AM, Owen Anderson
wrote:<br>
</div>
<blockquote cite="mid:8244049A-C7C0-44AF-AABE-14AEC6080623@mac.com"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<br>
<div>
<div>On Jun 11, 2014, at 11:29 PM, deadal nix <<a
moz-do-not-send="true" href="mailto:deadalnix@gmail.com">deadalnix@gmail.com</a>>
wrote:</div>
<br class="Apple-interchange-newline">
<blockquote type="cite"><span style="font-family: Helvetica;
font-size: 12px; font-style: normal; font-variant: normal;
font-weight: normal; letter-spacing: normal; line-height:
normal; orphans: auto; text-align: start; text-indent: 0px;
text-transform: none; white-space: normal; widows: auto;
word-spacing: 0px; -webkit-text-stroke-width: 0px; float:
none; display: inline !important;">It is gonna improve the
situation quite a lot for all frontend that use aggregate
loads (arguably, that is a bad practice, but that no reason
to stab people in the back when they do it anyway).</span><br>
</blockquote>
<br>
</div>
<div>I’m not sure I agree with that statement. If we don’t think
they should be used, not optimizing them is a good way to
discourage that. More generally, I’m concerned about how we
will ever get good test coverage of this code path, since we
don’t have any extant front ends that hit it.</div>
</blockquote>
I'm joining this discussion late, but a) why are aggregate loads bad
practice? Loading something like a small struct from memory with a
single load seems reasonable. b) There are frontends that are not
in tree. The fact that Deadal submitted the patch is a good hint
that *someone* cares. c) Is there a phabricator link for this
change? I can't find the original thread. <br>
<br>
Philip<br>
</body>
</html>