Skip to content

Conversation

omjavaid
Copy link
Contributor

This patch implements timeit timing tool found at tools/timeit.c for Windows.
It is required for compile time and run time calculations by the llvm-testsuite.

At this moment llvm testsuite tests can not be compiled with clang-cl MSVC combination
however we have also made required modifications to the cmake files to enable building
timeit on windows with clang-cl/cl.

In developer console with MSVC following command can be used to successfully build this code:

set CC=c:\work\llvm-dev\build\bin\clang-cl.exe
set CXX=c:\work\llvm-dev\build\bin\clang-cl.exe

cmake -G Ninja -DTEST_SUITE_SUBDIRS= ..\llvm-test-suite

Phabricator Review: https://reviews.llvm.org/D153263

@omjavaid omjavaid requested a review from mstorsjo November 14, 2023 12:13
This patch implements timeit timing tool found at tools/timeit.c for Windows.
It is required for compile time and run time calculations by the llvm-testsuite.

At this moment llvm testsuite tests can not be compiled with clang-cl MSVC combination
however we have also made required modifications to the cmake files to enable building
timeit on windows with clang-cl/cl.

In developer console with MSVC following command can be used to successfully build this code:

set CC=c:\work\llvm-dev\build\bin\clang-cl.exe
set CXX=c:\work\llvm-dev\build\bin\clang-cl.exe

cmake -G Ninja -DTEST_SUITE_SUBDIRS= ..\llvm-test-suite

Phabricator Review: https://reviews.llvm.org/D153263
Copy link
Member

@mstorsjo mstorsjo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, assuming that you've tested it on both Unix and Windows.

@omjavaid omjavaid merged commit be6bd27 into llvm:main Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants