[llvm-commits] Adding a builder for ExecutionEngines

[Reid Kleckner]

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.

Patch attached and linked:
http://codereview.appspot.com/91086/show

Thanks,
Reid
-------------- next part --------------
A non-text attachment was scrubbed...
Name: EngineBuilderPatch.diff
Type: text/x-diff
Size: 15684 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20090708/69e631db/attachment.diff>