[PATCH] D100442: [flang][msvc] Fix compilation of RuntimeGtest tests.

Wed Apr 14 02:03:47 PDT 2021

MehdiChinoune added inline comments.

================
Comment at: flang/unittests/RuntimeGTest/CMakeLists.txt:21
   FortranRuntime
+  ${llvm_libs}
 )
----------------
awarzynski wrote:
> This will basically add a new dependency for all configurations - how come this didn't fail before on other platforms? Also, there's a Flang buildbot worker that sets `LLVM_LINK_LLVM_DYLIB` to `On`: https://lab.llvm.org/buildbot/#/builders/135 and it's currently :green:. 
> 
> Separately, `Support` is added as a dependency for unit tests here: https://github.com/llvm/llvm-project/blob/main/llvm/cmake/modules/AddLLVM.cmake#L1452.
> 
> So, is this change really needed?
I don't know but the compiler doesn't recognize llvm::errs() `CrashHandlerFixture.cpp:19` Unless explicitly linked with `Support` component, I saw the same here https://github.com/llvm/llvm-project/blob/main/flang/unittests/Evaluate/CMakeLists.txt#L6

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D100442