Remove /nologo from windows build

[TyanNN]

Summary

/nologo emits error on new versions of rc.exe
PR#33493

[probinson]

I am mildly curious how much noise this adds to a build log.

cmake/modules/AddLLVM.cmake

675

Usually whitespace/formatting fixes are done separately.

1331

Usually whitespace/formatting fixes are done separately.

[TyanNN]

In D34521#789366, @probinson wrote:

I am mildly curious how much noise this adds to a build log.

Well, that's not really what i intended to do, it's just emacs save-hook that strips spaces at the end of the line, if it's necessary, i might disable it and update the patch

[TyanNN]

ping @chandlerc @bogner @mgorny

cmake/modules/AddLLVM.cmake

   endif()
 
   set_property(SOURCE ${resource_file}
-               PROPERTY COMPILE_FLAGS /nologo)
+               PROPERTY COMPILE_FLAGS)
   set_property(SOURCE ${resource_file}
                PROPERTY COMPILE_DEFINITIONS
                "RC_VERSION_FIELD_1=${ARG_VERSION_MAJOR}"

     # it forces Xcode to properly link the static library.
     list(APPEND ALL_FILES "${LLVM_MAIN_SRC_DIR}/cmake/dummy.cpp")
   endif()
-  
   if( EXCLUDE_FROM_ALL )
     add_executable(${name} EXCLUDE_FROM_ALL ${ALL_FILES})
   else()

   # magic. First we grab one of the types, and a type-specific path. Then from
   # the type-specific path we find the last occurrence of the type in the path,
   # and replace it with CMAKE_CFG_INTDIR. This allows the build step to be type
-  # agnostic again. 
+  # agnostic again.
   if(NOT ARG_OUTPUT_DIR)
     # If you're not overriding the OUTPUT_DIR, we can make the link relative in
     # the same directory.