Ah, I see this was addressed in 250386 - though might be nice to address across the whole project rather than this particular inclusion. Though maybe we’ve few enough inclusions that it’s not too bad.
I'd rather not have an extra flag to every compilation just to handle the
odd inclusion of windows.h. Most of LLVM should not be including
system-specific headers, and the parts that do should be small.
Can you instead always proxy-include windows.h? Force all LLVM inclusions of windows.h to go through lib/Support/Windows/WindowsSupport.h (move it to include/ instead) and add NOMINMAX. It already has WIN32_LEAN_AND_MEAN, you could even pile on with VC_EXTRALEAN.