<div dir="ltr"><div dir="ltr">Hi,</div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Em sex., 10 de dez. de 2021 às 12:37, Peter Smith <<a href="mailto:Peter.Smith@arm.com">Peter.Smith@arm.com</a>> escreveu:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">





<div lang="EN-GB" style="overflow-wrap: break-word;">
<div class="gmail-m_758101610381916520WordSection1">
<p class="MsoNormal"><span>Hello,<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span>My turn for the double post, my mail client stripped off the list on reply-all and I’d assumed it was llvm-dev. My apologies, resending back on cfe-dev<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span>This is likely down to the default triple and CPU that clang uses.<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span>An arm-linux-gnueabihf-gcc defaults to Thumb architecture v7-A which supports Thumb2. Whereas clang defaults to Arm state and architecture v4t which does not support Thumb2.<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span>On a relatively recent build of clang, but I think this goes back some way:<u></u><u></u></span></p>
<p class="MsoNormal"><span>clang --target=arm-linux-gnueabihf -mthumb -march=armv7-a -dM -E - < /dev/null | grep thumb<u></u><u></u></span></p>
<p class="MsoNormal"><span>#define __thumb2__ 1<u></u><u></u></span></p>
<p class="MsoNormal"><span>#define __thumb__ 1<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span>Can you give that a try?<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u></span></p></div></div></blockquote><div><br></div><div>Nice, it now defines them:</div><div><br></div><div>$ clang --target=arm-linux-gnueabihf -mthumb -march=armv7-a -dM -E - < /dev/null | grep thumb<br>#define __thumb2__ 1<br>#define __thumb__ 1<br></div><div><br></div><div>but does it mean that if I don't pass these arguments clang won't thumb2 code? Despite __ARM_ARCH_ISA_THUMB being set to 2?</div><div><br></div><div>Thank you</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div lang="EN-GB" style="overflow-wrap: break-word;"><div class="gmail-m_758101610381916520WordSection1"><p class="MsoNormal"><span> <u></u></span></p>
<p class="MsoNormal"><span>Peter<u></u><u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<p class="MsoNormal"><span><u></u> <u></u></span></p>
<div style="border-top:none;border-right:none;border-bottom:none;border-left:1.5pt solid blue;padding:0cm 0cm 0cm 4pt">
<div>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US">From:</span></b><span lang="EN-US"> cfe-dev <<a href="mailto:cfe-dev-bounces@lists.llvm.org" target="_blank">cfe-dev-bounces@lists.llvm.org</a>>
<b>On Behalf Of </b>Mikhail R. Gadelha via cfe-dev<br>
<b>Sent:</b> 10 December 2021 15:32<br>
<b>To:</b> clang mailing-list <<a href="mailto:cfe-dev@lists.llvm.org" target="_blank">cfe-dev@lists.llvm.org</a>><br>
<b>Subject:</b> [cfe-dev] Question about thumb2 define in clang for ARMv8<u></u><u></u></span></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">(Sorry if this is a double post, I sent this email a few days ago from an e-mail that's not subscribed to the mailing list and it seems to be stuck in moderation).<br clear="all">
<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Hi everyone,<br>
<br>
tl;dr: GCC defines __thumb2__ in ARMv8 and clang doesn't. Is that a bug? Is there a better way to check if thumb2 is supported? Maybe __ARM_ARCH_ISA_THUMB?<br>
<br>
~<br>
<br>
I recently started to work with Webkit in ARMv8 and decided to give clang a try building the javascript engine (called JSC) used by WebKit.<br>
<br>
However, it turns out the JIT was always disabled because of a CMake script that checks if the compiler defines either thumb2 or __thumb2__:<br>
<br>
<a href="https://github.com/WebKit/WebKit/blob/363701084a50c009bec41164cdb843df66d769d1/Source/cmake/OptionsCommon.cmake#L31" target="_blank">https://github.com/WebKit/WebKit/blob/363701084a50c009bec41164cdb843df66d769d1/Source/cmake/OptionsCommon.cmake#L31</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><br>
I've tested with some other versions of clang but none seem to define it: <a href="https://godbolt.org/z/WjGh3eTzv" target="_blank">
https://godbolt.org/z/WjGh3eTzv</a><br>
<br>
While __thumb2__ is defined in GCC:<br>
<br>
$ gcc -dM -E  main.c  | grep thumb<br>
#define __thumb2__ 1<br>
#define __thumb__ 1<br>
<br>
clang doesn't define __thumb__ nor __thumb2__.<br>
<br>
I managed to get clang with __thumb2__ defined by:<br>
1. Changing the triplet: I eventually got a lot of compilation errors and gave up.<br>
2. Added "-D__thumb__=1" to CMAKE_CXX_FLAGS :P and managed to fully build JSC. There were some regressions in a place I was investigating a bug but nothing major.<br>
<br>
Anyway, I guess my questions are:<br>
1. Is there a reason why __thumb2__ isn't defined in clang? Do we want it to be defined in clang?<br>
2. Should we change WebKit to check if "__ARM_ARCH_ISA_THUMB" is 2 as per [0], instead of __thumb2__? (both GCC and clang seem to define it to 2 in the hardware I'm testing).<br>
<br>
Thanks in advance.<br>
Mikhail.<br>
<br>
[0] <a href="https://github.com/llvm/llvm-project/blob/8ee55ab6c06ec9f769f1086e3761c92286d56339/clang/lib/Basic/Targets/ARM.cpp#L709" target="_blank">
https://github.com/llvm/llvm-project/blob/8ee55ab6c06ec9f769f1086e3761c92286d56339/clang/lib/Basic/Targets/ARM.cpp#L709</a><u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<p class="MsoNormal">-- <u></u><u></u></p>
<div>
<div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Mikhail R. Gadelha.<u></u><u></u></p>
</div>
</div>
</div>
</div>
</div>
</div>
</div>

</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><br></div><div>Mikhail Ramalho.</div></div></div></div>