<div dir="ltr"><div dir="ltr"><div dir="ltr"><div dir="ltr">Hi Nicolas,</div><br><div class="gmail_quote"><div dir="ltr">On Fri, Jan 4, 2019 at 3:39 AM Nicolas Cellier <<a href="mailto:nicolas.cellier.aka.nice@gmail.com">nicolas.cellier.aka.nice@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"> <div dir="ltr"><div dir="ltr"><div dir="ltr"><div>We have this code<br></div><div><br></div><div>voidLongRunningPrimitive: reason <br></div><div>    ...<br></div><div>    <inline: false><br>    longRunningPrimitiveCheckMethod := nil.<br>    longRunningPrimitiveCheckSequenceNumber = 0. "not strictly necessary but prevents this being inlined into checkForLongRunningPrimitive"<br>   ...</div><div><br></div><div>which generates</div><div><br></div><div>static void NoDbgRegParms<br>voidLongRunningPrimitive(char *reason)<br>{   DECL_MAYBE_SQ_GLOBAL_STRUCT<br>    GIV(longRunningPrimitiveCheckMethod) = null;<br>    GIV(longRunningPrimitiveCheckSequenceNumber) == 0;</div><div>   ...</div><div><br></div><div>and a compiler warning suggesting to replace == with =</div><div>So it seems to effectively be a hack, but it seems fragile.<br></div><div>Couldn't we remove this line (don't we have inline: #never for that purpose?).<br></div></div></div></div></blockquote><div><br></div><div>Yes.  When I wrote the code we didn't have inline: #never.  inline: #never is a much better solution.</div><div><br></div></div><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-size:small;border-collapse:separate"><div>_,,,^..^,,,_<br></div><div>best, Eliot</div></span></div></div></div></div></div></div>