[PATCH] D23654: [CMake] Rename back SIMPLE_SOURCE to compile as C++

[Saleem Abdulrasool via llvm-commits]

compnerd requested changes to this revision.
compnerd added a comment.
This revision now requires changes to proceed.

Sorry, forgot to respond to @beanz' question of how does it fail.  When building for ARM:

  error: undefined reference to '__aeabi_unwind_cpp_pr0'

This is provided by libunwind.  There are a few different providers for the personality routine, but that requires that you build those, which depend on the builtins, so we can't really guarantee that it exists (bootstrapping).

>From your description, it sounds like you want to ensure that the C++ compiler is valid?  Why not add a secondary test that does a *compile only* test for C++ support without depending on C++ headers?  Something like a class definition should be sufficient for that.

Alternatively, we could try `-fno-exceptions -fno-unwind-tables` as part of the compile options for the test target.


https://reviews.llvm.org/D23654