[PATCH] D60043: [FileCheck] Fix FileCheck.cpp compilation on Solaris

Rainer Orth via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 2 06:06:00 PDT 2019


ro marked an inline comment as done.
ro added inline comments.


================
Comment at: include/llvm/Support/FileCheck.h:22
 #include <map>
+#include <cmath>
 
----------------
jdenny wrote:
> Does this need to be processed by all includers?  Shouldn't it be sufficient in FileCheck.cpp?
FileCheck.cpp didn't include any non-LLVM headers before,
therefore I placed the include in a common header that did.


Repository:
  rL LLVM

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

https://reviews.llvm.org/D60043





More information about the llvm-commits mailing list