[PATCH] D61243: Check build-type dependent variables for -flto as well.

[Florian Hahn via Phabricator via llvm-commits]

fhahn added a comment.

In D61243#1481819 <https://reviews.llvm.org/D61243#1481819>, @lebedev.ri wrote:

> Two thoughts:
>
> 1. Why is `LDFLAGS` being set conditionally, is there a warning otherwise?


I think it is set conditionally, because the selected linker exe may not support it. When doing LTO  and saving stats, the assumption is that people are using clang for linking.

> 2. I wonder, why it's `{C,CXX,LD}FLAGS`, and not `CMAKE_*_FLAGS` directly.

Not sure, TBH.


Repository:
  rT test-suite

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

https://reviews.llvm.org/D61243