================
@@ -5865,6 +5865,7 @@ def print_enabled_extensions : Flag<["-", "--"], 
"print-enabled-extensions">,
 def : Flag<["-"], "mcpu=help">, Alias<print_supported_cpus>;
 def : Flag<["-"], "mtune=help">, Alias<print_supported_cpus>;
 def time : Flag<["-"], "time">,
+  Visibility<[ClangOption, CLOption, DXCOption, FlangOption]>,
----------------
tarunprabhu wrote:

I don't see any reason why it should not work on Windows. But there was a 
failing Windows buildkite, so I temporarily disabled the test. I will try again 
to see if it was a transient issue. If the issue still persists, I will add a 
comment indicating that the option should work on Windows, but that the 
original author did not have a means to test it and, therefore, disabled it. 
Since some of the tests require a full toolchain to be present, @MaskRay 
suggested being conservative about where the tests were run.

I have generally kept `CLOption` and `DXCOption` when the original did not 
explicitly specify anything for compatibility.

https://github.com/llvm/llvm-project/pull/109165
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to