Correctness fix should be here:<br><div><br></div><div><div><span class="Apple-tab-span" style="white-space:pre"> </span>M<span class="Apple-tab-span" style="white-space:pre">   </span>bindings/go/llvm/executionengine.go</div><div><span class="Apple-tab-span" style="white-space:pre">  </span>M<span class="Apple-tab-span" style="white-space:pre">   </span>bindings/go/llvm/executionengine_test.go</div><div>Committed r222805</div></div><div><br></div><div>I'll see if I can work up some code to wrap the other methods.</div><div><br></div><br><div class="gmail_quote">On Tue Nov 25 2014 at 6:44:13 PM Eric Christopher <<a href="mailto:echristo@gmail.com">echristo@gmail.com</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="gmail_quote">On Tue Nov 25 2014 at 6:42:53 PM Peter Collingbourne <<a href="mailto:peter@pcc.me.uk" target="_blank">peter@pcc.me.uk</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Wed, Nov 26, 2014 at 02:27:46AM -0000, Eric Christopher wrote:<br>
> +func SetMCJITOptimizationLevel(<u></u>opti<u></u>ons MCJITCompilerOptions, level uint) {<br>
> +     options.C.OptLevel = C.uint(level)<br>
> +}<br>
<br>
This function has no effect, as 'options' is passed by value.<br>
<br>
This would be better as a method on the MCJITCompilerOptions type.<br>
The signature should look like this in order to pass 'options' by reference:<br>
<br>
func (options *MCJITCompilerOptions) SetOptLevel(level uint) { ... }<br>
<br>
We should also have setters for the other properties.<br>
<br></blockquote><div><br></div></div><div class="gmail_quote"><div>Cool, I wasn't sure if you wanted setters on the type or free functions. I'll move things around and play. Also, thanks for the review :)</div><div><br></div><div>-eric</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks,</blockquote></div><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
--<br>
Peter<br>
</blockquote></div></blockquote></div>