[PATCH] D55056: [CMake] Default options for faster executables on MSVC

Alexandre Ganea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 3 11:22:01 PST 2018


aganea added a comment.

In D55056#1317117 <https://reviews.llvm.org/D55056#1317117>, @rnk wrote:

> In D55056#1315675 <https://reviews.llvm.org/D55056#1315675>, @aganea wrote:
>
> > In that case can’t we just keep Debug and Release+PDB, and drop the two others?
>
>
> I guess I'm not sure what you mean. Do you mean, drop them, as in find a way to remove them from the list of default configurations so that they don't display in IDEs as a configuration option? That makes sense. But I imagine that IDE users would want some configuration for "Release+PDB" that doesn't require re-running cmake, and RelWithDebInfo seems like the right name for it.


The other way around: why would you keep Release without debug info?


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D55056/new/

https://reviews.llvm.org/D55056





More information about the llvm-commits mailing list