Re: JIT compiling with LLVM v9.0
Andres Freund <andres@anarazel.de>
Hi, On 2018-01-29 10:28:18 -0800, Jeff Davis wrote: > OK. How about this: are you open to changes that move us in the > direction of extensibility later? (By this I do *not* mean imposing a > bunch of requirements on you... either small changes to your patches > or something part of another commit.) I'm good with that. > Or are you determined that this always should be a part of core? I do think JIT compilation should be in core, yes. And after quite some looking around that currently means either using LLVM or building our own from scratch, and the latter doesn't seem attractive. But that doesn't mean there *also* can be extensibility. If somebody wants to experiment with a more advanced version of JIT compilation, develop a gcc backed version (which can't be in core due to licensing), ... - I'm happy to provide hooks that only require a reasonable effort and don't affect the overall stability of the system (i.e. no callback from PostgresMain()'s sigsetjmp() block). > I don't want to stand in your way, but I am also hesitant to dive head > first into LLVM and not look back. Postgres has always been lean, fast > building, and with few dependencies. It's an optional dependency, and it doesn't increase build time that much... If we were to move the llvm interfacing code to a .so, there'd not even be a packaging issue, you can just package that .so separately and get errors if somebody tries to enable LLVM without that .so being installed. > In other words, are you "strongly against [extensbility being a > requirement for the first commit]" or "strongly against [extensible > JIT]"? I'm strongly against there not being an in-core JIT. I'm not at all against adding APIs that allow to do different JIT implementations out of core. > If the source for functions is in the catalog, we could build the > bitcode at runtime and still do the inlining. We wouldn't need to do > anything at build time. (Again, this would be "cool stuff for the > future", I am not asking you for it now.) Well, the source would require an actual compiler around. And the inlining *just* for the function code itself isn't actually that interesting, you e.g. want to also be able to Greetings, Andres Freund
Commits
-
Improve JIT docs.
- fb604780114c 11.0 landed
-
Add documentation for the JIT feature.
- e6c039d13e16 11.0 landed
-
Add EXPLAIN support for JIT.
- 1f0c6a9e7dca 11.0 landed
-
Add inlining support to LLVM JIT provider.
- 9370462e9a79 11.0 landed
-
JIT tuple deforming in LLVM JIT provider.
- 32af96b2b118 11.0 landed
-
Add FIELDNO_* macro designating offset into structs required for JIT.
- 7ced1d124728 11.0 landed
-
Add expression compilation support to LLVM JIT provider.
- 2a0faed9d702 11.0 landed
-
Expand list of synchronized types and functions in LLVM JIT provider.
- fb46ac26fe49 11.0 landed
-
Add helpers for emitting LLVM IR.
- 7ec0d80c0508 11.0 landed
-
Basic planner and executor integration for JIT.
- cc415a56d09a 11.0 landed
-
Debugging and profiling support for LLVM JIT provider.
- 250bca7fc145 11.0 landed
-
Support for optimizing and emitting code in LLVM JIT provider.
- b96d550eb03c 11.0 landed
-
Add file containing extensions of the LLVM C API.
- 31bc604e0b74 11.0 landed
-
Basic JIT provider and error handling infrastructure.
- 432bb9e04da4 11.0 landed
-
Add configure infrastructure (--with-llvm) to enable LLVM support.
- 5b2526c83832 11.0 landed
-
Add C++ support to configure.
- 6869b4f25847 11.0 landed
-
Add PGAC_PROG_VARCC_VARFLAGS_OPT autoconf macro.
- 3de04e4ed12d 11.0 landed
-
Fix VM buffer pin management in heap_lock_updated_tuple_rec().
- 0b1d1a038bab 11.0 cited
-
Allow tupleslots to have a fixed tupledesc, use in executor nodes.
- ad7dbee368a7 11.0 landed
-
Expression evaluation based aggregate transition invocation.
- 69c3936a1499 11.0 landed
-
Perform slot validity checks in a separate pass over expression.
- b40933101ca6 11.0 landed
-
Rely on executor utils to build targetlist for DML RETURNING.
- 4717fdb14cf0 11.0 landed
-
Refer to OS X as "macOS", except for the port name which is still "darwin".
- da6c4f6ca88d 10.0 cited