[llvm-branch-commits] [llvm] 3ca8b23 - Fix spurious warning in ExtensionDependencies.inc [nfc]

Tom Stellard via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Wed Jun 17 17:36:11 PDT 2020


Author: serge-sans-paille
Date: 2020-06-17T16:57:30-07:00
New Revision: 3ca8b23cf5b9afc4ea05ac0533fbb874c5660e1d

URL: https://github.com/llvm/llvm-project/commit/3ca8b23cf5b9afc4ea05ac0533fbb874c5660e1d
DIFF: https://github.com/llvm/llvm-project/commit/3ca8b23cf5b9afc4ea05ac0533fbb874c5660e1d.diff

LOG: Fix spurious warning in ExtensionDependencies.inc [nfc]

(cherry picked from commit 37309fb02f60557f18971dc575904c0fc56c91ab)

Added: 
    

Modified: 
    llvm/cmake/modules/AddLLVM.cmake

Removed: 
    


################################################################################
diff  --git a/llvm/cmake/modules/AddLLVM.cmake b/llvm/cmake/modules/AddLLVM.cmake
index bd22353b6cfc..f5a1b0d6f238 100644
--- a/llvm/cmake/modules/AddLLVM.cmake
+++ b/llvm/cmake/modules/AddLLVM.cmake
@@ -972,12 +972,12 @@ function(process_llvm_pass_plugins)
               const char* Name;\n\
               const char* RequiredLibraries[1 + 1 + ${llvm_plugin_max_deps_length}];\n\
            };\n\
-           std::array<ExtensionDescriptor, ${llvm_static_extension_count}>  AvailableExtensions{\n")
+           std::array<ExtensionDescriptor, ${llvm_static_extension_count}> AvailableExtensions{\n")
 
       foreach(llvm_extension ${LLVM_STATIC_EXTENSIONS})
         get_property(llvm_plugin_deps TARGET ${llvm_extension} PROPERTY LINK_LIBRARIES)
 
-        file(APPEND "${ExtensionDeps}.tmp" "{\"${llvm_extension}\", {")
+        file(APPEND "${ExtensionDeps}.tmp" "{{\"${llvm_extension}\", {")
         foreach(llvm_plugin_dep ${llvm_plugin_deps})
             # Turn library dependency back to component name, if possible.
             # That way llvm-config can avoid redundant dependencies.
@@ -991,7 +991,7 @@ function(process_llvm_pass_plugins)
         endforeach()
 
         # Self + mandatory trailing null, because the number of RequiredLibraries 
diff ers between extensions.
-        file(APPEND "${ExtensionDeps}.tmp" \"${llvm_extension}\", "nullptr}},\n")
+        file(APPEND "${ExtensionDeps}.tmp" \"${llvm_extension}\", "nullptr}}},\n")
       endforeach()
       file(APPEND "${ExtensionDeps}.tmp" "};\n")
 


        


More information about the llvm-branch-commits mailing list