[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