[llvm] [LLVM][CMake][MSVC] Install PDBs alongside executables (PR #120683)
Miguel A. Arroyo via llvm-commits
llvm-commits at lists.llvm.org
Tue Jan 14 11:43:38 PST 2025
mayanez wrote:
> I think that you should use LLVM_ENABLE_PDB instead of MSVC for the condition.
That's a good callout! I can make the change for that.
> I’d rather prefer publishing them on a symbol server, and pull them on-demand instead
While orthogonal, wouldn't having them in the `install` directory simplify having to fetch all the PDBs to be deployed?
> I think that @zmodem should verify that the installer would not package up the PDBs.
Yes, if the installer doesn't pick it up that would be ideal. Otherwise, I can take a look at that next.
https://github.com/llvm/llvm-project/pull/120683
More information about the llvm-commits
mailing list