[llvm] [CMAKE][llvm-libraries] Add Precompiled Headers option to improve build times (PR #91755)
    Volodymyr Sapsai via llvm-commits 
    llvm-commits at lists.llvm.org
       
    Mon Jul  8 19:39:23 PDT 2024
    
    
  
vsapsai wrote:
Ok, looks like there are no objections to the usage of precompiled headers in general. So now I'll look at the technical side of the change. Please note that I'm not good with CMake and might ask some stupid questions.
I've tried your patch locally and hitting the error
```
-- LLVM Lib Precompiled Headers are enabled
CMake Error at cmake/modules/LLVMPrecompiledHeaders.cmake:2 (get_property):
  get_property DIRECTORY scope provided but requested directory was not
  found.  This could be because the directory argument was invalid or, it is
  valid but has not been processed yet.
Call Stack (most recent call first):
  cmake/modules/LLVMPrecompiledHeaders.cmake:13 (get_all_targets_recursive)
  cmake/modules/LLVMPrecompiledHeaders.cmake:25 (get_all_targets)
  cmake/modules/LLVMPrecompiledHeaders.cmake:69 (add_llvm_subdir_pch)
  CMakeLists.txt:1419 (llvm_lib_precompiled_headers)
```
I haven't investigated the issue yet but maybe it is something you've seen, so I've decided to check first if it is a known problem.
https://github.com/llvm/llvm-project/pull/91755
    
    
More information about the llvm-commits
mailing list