[PATCH] D84459: [llvm][NFC] Don't use llvm/Config/config.h in .h files

[Nico Weber via Phabricator via llvm-commits]

thakis added a comment.

I wouldn't worry about the Windows header. It's some pre-existing thing and nobody seems to care.



================
Comment at: llvm/include/llvm/Analysis/InlineAdvisor.h:216
                           std::function<bool(CallBase &)> GetDefaultAdvice);
-#endif
 
----------------
Hm, maybe it's better to move the other define to llvm-config and make these APIs available only if the config header advertises their existence? It seems a bit nicer for external clients if they can check at build time instead of at runtime maybe.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D84459