Closed iynehz closed 5 years ago
This is regarding #1. IMHO this provides a more generic way to detect clang than what's implemented in Inline::CPP now.
But Inline::CPP might need a change https://github.com/daoswald/Inline-CPP/blob/bf385186b513092539cd53ad9038e1184700322a/inc/ILCPPConfig/CompilerGuess.pm#L30 as Clang is also considered gcc (by existing ExtUtils::CppGuess 0.11).
An alternative way to do it could be that is_gcc() still returns true for clang, but is_clang() returns false for gcc. So clang "is" gcc but gcc "is not" clang.
@leont @daoswald Does this seem like the right way forward to you?
Hi @stphnlyd, could you try the new 0.13 and see if it works? If not, please reopen this issue (and resolve the conflicts that now exist).
This is regarding #1. IMHO this provides a more generic way to detect clang than what's implemented in Inline::CPP now.
But Inline::CPP might need a change https://github.com/daoswald/Inline-CPP/blob/bf385186b513092539cd53ad9038e1184700322a/inc/ILCPPConfig/CompilerGuess.pm#L30 as Clang is also considered gcc (by existing ExtUtils::CppGuess 0.11).
An alternative way to do it could be that is_gcc() still returns true for clang, but is_clang() returns false for gcc. So clang "is" gcc but gcc "is not" clang.