This is an archive of the discontinued LLVM Phabricator instance.

[clang][lit] Allow test cases to use the compiler that are used to compile Clang
ClosedPublic

Authored by OikawaKirie on Mar 18 2021, 8:32 PM.

Details

Summary

Required by D83660. Test cases may want to use the host compiler to compile some mocks for the test case. This patch adds two substitutions %host_cc and %host_cxx to use the host compilers set via variable CMAKE_C_COMPILER and CMAKE_CXX_COMPILER.

Diff Detail

Event Timeline

OikawaKirie requested review of this revision.Mar 18 2021, 8:32 PM
OikawaKirie created this revision.
steakhal accepted this revision.Mar 19 2021, 1:43 AM

Oh, wait a minute. The config.host_cxx was already there.
Where is the symmetry :D It looks much better now, thanks.

Regarding D83660, I'm really looking forward to it. It's a shame that it takes so long to land it.

This revision is now accepted and ready to land.Mar 19 2021, 1:43 AM

Oh, wait a minute. The config.host_cxx was already there.
Where is the symmetry :D It looks much better now, thanks.

Sounds not bad if it matters to you :P

Regarding D83660, I'm really looking forward to it. It's a shame that it takes so long to land it.

Please commit this patch on my behalf (Ella Ma <alansnape3058@gmail.com>), so that I can continue with D83660 with your mocked solver.

Please commit this patch on my behalf (Ella Ma <alansnape3058@gmail.com>), so that I can continue with D83660 with your mocked solver.

Oh, I forgot about this one. My apologies.