Correct some typos in the new JIT code.

Author: Thomas Munro

diff --git a/src/backend/jit/llvm/llvmjit.c b/src/backend/jit/llvm/llvmjit.c
index d73237d002d6da52821ab87fb7910e89f003781d..bbf30b31552a30150d51e36c9fafde4ea9d08afd 100644 (file)
--- a/src/backend/jit/llvm/llvmjit.c
+++ b/src/backend/jit/llvm/llvmjit.c
@@ -247,7 +247,7 @@ llvm_get_function(LLVMJitContext *context, const char *funcname)
 
    /*
     * If there is a pending / not emitted module, compile and emit now.
-    * Otherwise we migh not find the [correct] function.
+    * Otherwise we might not find the [correct] function.
     */
    if (!context->compiled)
    {
@@ -266,7 +266,7 @@ llvm_get_function(LLVMJitContext *context, const char *funcname)
 
        addr = 0;
        if (LLVMOrcGetSymbolAddressIn(handle->stack, &addr, handle->orc_handle, funcname))
-           elog(ERROR, "failed to lookup symbol \"%s\"", funcname);
+           elog(ERROR, "failed to look up symbol \"%s\"", funcname);
        if (addr)
            return (void *) (uintptr_t) addr;
    }
@@ -280,11 +280,11 @@ llvm_get_function(LLVMJitContext *context, const char *funcname)
        return (void *) (uintptr_t) addr;
 #else
    if (LLVMOrcGetSymbolAddress(llvm_opt0_orc, &addr, funcname))
-       elog(ERROR, "failed to lookup symbol \"%s\"", funcname);
+       elog(ERROR, "failed to look up symbol \"%s\"", funcname);
    if (addr)
        return (void *) (uintptr_t) addr;
    if (LLVMOrcGetSymbolAddress(llvm_opt3_orc, &addr, funcname))
-       elog(ERROR, "failed to lookup symbol \"%s\"", funcname);
+       elog(ERROR, "failed to look up symbol \"%s\"", funcname);
    if (addr)
        return (void *) (uintptr_t) addr;
 #endif                         /* LLVM_VERSION_MAJOR */
@@ -540,7 +540,7 @@ llvm_compile_module(LLVMJitContext *context)
        if (LLVMOrcAddEagerlyCompiledIR(compile_orc, &orc_handle, smod,
                                        llvm_resolve_symbol, NULL))
        {
-           elog(ERROR, "failed to jit module");
+           elog(ERROR, "failed to JIT module");
        }
        LLVMOrcDisposeSharedModuleRef(smod);
    }
@@ -847,7 +847,7 @@ llvm_resolve_symbol(const char *symname, void *ctx)
    char       *modname;
 
    /*
-    * OSX prefixes all object level symbols with an underscore. But neither
+    * macOS prefixes all object level symbols with an underscore. But neither
     * dlsym() nor PG's inliner expect that. So undo.
     */
 #if defined(__darwin__)

diff --git a/src/backend/jit/llvm/llvmjit_error.cpp b/src/backend/jit/llvm/llvmjit_error.cpp
index edc1c479d01c27f87f3f52f7ba8baf974baea723..a2bdfe3fb87a2e6570b06c6e9dcc665cf5f894ad 100644 (file)
--- a/src/backend/jit/llvm/llvmjit_error.cpp
+++ b/src/backend/jit/llvm/llvmjit_error.cpp
@@ -4,7 +4,7 @@
  *   LLVM error related handling that requires interfacing with C++
  *
  * Unfortunately neither (re)setting the C++ new handler, nor the LLVM OOM
- * handler are exposed to C. Therefore this file wraps the necesary code.
+ * handler are exposed to C. Therefore this file wraps the necessary code.
  *
  * Copyright (c) 2016-2018, PostgreSQL Global Development Group
  *
@@ -39,12 +39,12 @@ static void fatal_llvm_error_handler(void *user_data, const std::string& reason,
  *
  * This is necessary for LLVM as LLVM's error handling for such cases
  * (exit()ing, throwing std::bad_alloc() if compiled with exceptions, abort())
- * isn't compatible with postgres error handling.  Thus in section where LLVM
+ * isn't compatible with postgres error handling.  Thus in sections where LLVM
  * code, not LLVM generated functions!, is executing, standard new, LLVM OOM
  * and LLVM fatal errors (some OOM errors masquerade as those) are redirected
  * to our own error handlers.
  *
- * These error handlers FATAL, because there's no reliable way from within
+ * These error handlers use FATAL, because there's no reliable way from within
  * LLVM to throw an error that's guaranteed not to corrupt LLVM's state.
  *
  * To avoid disturbing extensions using C++ and/or LLVM, these handlers are

diff --git a/src/backend/jit/llvm/llvmjit_expr.c b/src/backend/jit/llvm/llvmjit_expr.c
index 2074b067bab90afaf44bbe5b1435851970f3f97c..0d8c2bd2fb063822ca001bbc71dac835c3c9e8e2 100644 (file)
--- a/src/backend/jit/llvm/llvmjit_expr.c
+++ b/src/backend/jit/llvm/llvmjit_expr.c
@@ -1984,7 +1984,7 @@ llvm_compile_expr(ExprState *state)
                                isnull;
 
                    /*
-                    * At this point aggref->aggno is not yet set (it's setup
+                    * At this point aggref->aggno is not yet set (it's set up
                     * in ExecInitAgg() after initializing the expression). So
                     * load it from memory each time round.
                     */
@@ -2020,7 +2020,7 @@ llvm_compile_expr(ExprState *state)
 
                    /*
                     * At this point aggref->wfuncno is not yet set (it's
-                    * setup in ExecInitWindowAgg() after initializing the
+                    * set up in ExecInitWindowAgg() after initializing the
                     * expression). So load it from memory each time round.
                     */
                    v_wfuncnop = l_ptr_const(&wfunc->wfuncno,

diff --git a/src/backend/jit/llvm/llvmjit_types.c b/src/backend/jit/llvm/llvmjit_types.c
index ad29bafa8f63f36e8b639d9b250a2fa171522703..0e06160c2aa6e8839af2f530c6c44c98c813c200 100644 (file)
--- a/src/backend/jit/llvm/llvmjit_types.c
+++ b/src/backend/jit/llvm/llvmjit_types.c
@@ -8,7 +8,7 @@
  * low chance of definitions getting out of sync, this file lists types and
  * functions that directly need to be accessed from LLVM.
  *
- * When LlVM is first used in a backend, a bitcode version of this file, will
+ * When LLVM is first used in a backend, a bitcode version of this file will
  * be loaded. The needed types and signatures will be stored into Struct*,
  * Type*, Func* variables.
  *