[PATCH] D34136: [Solaris] replace Solaris.h hack with a set of better hacks

[Fedor Sergeev via Phabricator via llvm-commits]

fedor.sergeev added a comment.

Verified that sys/regset.h works by adding
+#include <sys/regset.h>
+int ES = 1;
into one of lib/CodeGen sources.

Btw, there is one catch here, gcc reports a pedantic warning on include_next:

  In file included from  llvm/lib/CodeGen/LivePhysRegs.cpp:16:0:
  llvm/include/llvm/Support/Solaris/sys/regset.h:18:2: warning: #include_next is a GCC extension
   #include_next <sys/regset.h>

I can silence this warning by adding
#pragma GCC system_header

into sys/regset.h interposing header, but I'm not sure whats the policy for using GCC pragmas in LLVM project...


https://reviews.llvm.org/D34136