[clang] [flang] [clang][flang] Support -time in both clang and flang (PR #109165)
Tarun Prabhu via cfe-commits
cfe-commits at lists.llvm.org
Tue Oct 15 06:46:47 PDT 2024
================
@@ -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
More information about the cfe-commits
mailing list