[PATCH] D45558: [test-suite] Save stats for LTO step too.

[Matthias Braun via Phabricator via llvm-commits]

MatzeB added inline comments.


================
Comment at: test-suite/trunk/CMakeLists.txt:198
+  # Collect stats for LTO step too.
+  if (${CMAKE_C_FLAGS} MATCHES ".*-flto.*" AND
+      ${CMAKE_CXX_FLAGS} MATCHES ".*-flto.*")
----------------
fhahn wrote:
> MatzeB wrote:
> > You could just append the flag to LDFLAHS unconditionally now, can you? (Or do nothing at all, I think currently we append cflags to all linker invocations too; which admittedly is bad style but unlikely to change...
> Ah I saw your comment too late, sorry! I think the main reason I originally added it conditionally is to avoid "unused argument" warnings when not doing LTO. What do you think?
I only added the comments after you committed so no worries. But I still think that adding all this after you did the changes for working without `-Wl,plugin` are not necessary anymore, maybe try removing them and if things still work remove this?


Repository:
  rL LLVM

https://reviews.llvm.org/D45558