[llvm-commits] Adding a builder for ExecutionEngines

[Eric Christopher]

On Jul 8, 2009, at 4:35 PM, Reid Kleckner wrote:

> In the last patch I was working on, Evan asked if I could add a
> builder for ExecutionEngines, since the current API call is kind of
> unwieldy.  It has four optional arguments, and a fifth if you count
> the JIT memory manager.  I attempted to unify this all into just one
> EngineBuilder class, which makes it so you only have to specify the
> arguments you want to control with chained setter methods.
>
> Does this seem like a better API?  Please review.

I definitely like the idea, any hope of getting a patch after  
Jeffrey's comments?

-eric