[llvm-commits] [llvm] r76276 - in /llvm/trunk: bindings/ocaml/executionengine/ docs/ docs/tutorial/ examples/BrainF/ examples/Fibonacci/ examples/HowToUseJIT/ examples/Kaleidoscope/ examples/ParallelJIT/ include/llvm/ExecutionEngine/ lib/Executio

[Chris Lattner]

On Jul 18, 2009, at 9:20 AM, Reid Kleckner wrote:

>> Why put these in a separate namespace? Why not just make a public  
>> enum
>> inside EngineBuilder? (I haven't been following the JIT API
>> discussions, so ignore me if this has already been hashed out)
>
> I was just copying what had been done elsewhere, for example
> CodeGenOpt::Type/Default/None/etc.  I can change it back if you like.

I prefer having the namespace, it avoids name pollution of llvm:: scope.

-Chris