Fix a few typos in JIT comments and README
authorDavid Rowley <[email protected]>
Thu, 20 Aug 2020 21:33:56 +0000 (09:33 +1200)
committerDavid Rowley <[email protected]>
Thu, 20 Aug 2020 21:33:56 +0000 (09:33 +1200)
Reviewed-by: Abhijit Menon-Sen
Reviewed-by: Andres Freund
Discussion: https://postgr.es/m/CAApHDvobgmCs6CohqhKTUf7D8vffoZXQTCBTERo9gbOeZmvLTw%40mail.gmail.com
Back-through: 11, where JIT was added

src/backend/jit/README
src/include/jit/llvmjit_emit.h

index e2fac8558e8e37d29d0d2e57ed0170e8c2eabfcf..5427bdf2153ff8c581e47e58ff6bcead1921595c 100644 (file)
@@ -10,11 +10,11 @@ SQL expressions to evaluate an SQL predicate like WHERE a.col = 3, it
 is possible to generate a function than can be natively executed by
 the CPU that just handles that expression, yielding a speedup.
 
-That this is done at query execution time, possibly even only in cases
-where the relevant task is done a number of times, makes it JIT,
-rather than ahead-of-time (AOT). Given the way JIT compilation is used
-in PostgreSQL, the lines between interpretation, AOT and JIT are
-somewhat blurry.
+This is JIT, rather than ahead-of-time (AOT) compilation, because it
+is done at query execution time, and perhaps only in cases where the
+relevant task is repeated a number of times. Given the way JIT
+compilation is used in PostgreSQL, the lines between interpretation,
+AOT and JIT are somewhat blurry.
 
 Note that the interpreted program turned into a native program does
 not necessarily have to be a program in the classical sense. E.g. it
@@ -99,7 +99,7 @@ Lifetimes of JITed functions are managed via JITContext. Exactly one
 such context should be created for work in which all created JITed
 function should have the same lifetime. E.g. there's exactly one
 JITContext for each query executed, in the query's EState.  Only the
-release of an JITContext is exposed to the provider independent
+release of a JITContext is exposed to the provider independent
 facility, as the creation of one is done on-demand by the JIT
 implementations.
 
@@ -231,7 +231,7 @@ needs to be referenced as an offset to one block of memory stored in
 an ExprState, rather than absolute pointers into memory.
 
 Once that is addressed, adding an LRU cache that's keyed by the
-generated LLVM IR will allow to use optimized functions even for
+generated LLVM IR will allow the usage of optimized functions even for
 faster queries.
 
 A longer term project is to move expression compilation to the planner
index 1a7d6db7259e0341cafe5b4096cd98bd845c7618..3142df608b3c654333e6cbf9a6749d29dc1bc019 100644 (file)
@@ -1,6 +1,6 @@
 /*
  * llvmjit_emit.h
- *   Helpers to make emitting LLVM IR a it more concise and pgindent proof.
+ *   Helpers to make emitting LLVM IR a bit more concise and pgindent proof.
  *
  * Copyright (c) 2018-2020, PostgreSQL Global Development Group
  *