<p>Damien Zammit has uploaded this change for <strong>review</strong>.</p><p><a href="https://review.coreboot.org/21358">View Change</a></p><pre style="font-family: monospace,monospace; white-space: pre-wrap;">arm/libgcc/lib1funcs.S: Workaround for clang<br><br>clang does not understand `movnes` and `subges` opcodes<br>so use the corresponding opcodes it knows about<br><br>Change-Id: I2150cc14f8a0ce7f819f29c49cb00c695448bc10<br>Signed-off-by: Damien Zammit <damien@zamaudio.com><br>---<br>M src/arch/arm/libgcc/lib1funcs.S<br>1 file changed, 8 insertions(+), 0 deletions(-)<br><br></pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">git pull ssh://review.coreboot.org:29418/coreboot refs/changes/58/21358/1</pre><pre style="font-family: monospace,monospace; white-space: pre-wrap;">diff --git a/src/arch/arm/libgcc/lib1funcs.S b/src/arch/arm/libgcc/lib1funcs.S<br>index a642185..6ee1feb 100644<br>--- a/src/arch/arm/libgcc/lib1funcs.S<br>+++ b/src/arch/arm/libgcc/lib1funcs.S<br>@@ -92,7 +92,11 @@<br>       subhs   \dividend, \dividend, \divisor, lsr #3<br>        orrhs   \result,   \result,   \curbit,  lsr #3<br>        cmp     \dividend, #0                   @ Early termination?<br>+#if IS_ENABLED(CONFIG_COMPILER_GCC)<br>    movnes  \curbit,   \curbit,  lsr #4     @ No, any more bits to do?<br>+#else<br>+   lsrsne  \curbit,   \curbit,  #4         @ No, any more bits to do?<br>+#endif<br>   movne   \divisor,  \divisor, lsr #4<br>   bne     1b<br> <br>@@ -178,7 +182,11 @@<br>   subhs   \dividend, \dividend, \divisor, lsr #3<br>        cmp     \dividend, #1<br>         mov     \divisor, \divisor, lsr #4<br>+#if IS_ENABLED(CONFIG_COMPILER_GCC)<br>      subges  \order, \order, #4<br>+#else<br>+   subsge  \order, \order, #4<br>+#endif<br>   bge     1b<br> <br>         tst     \order, #3<br></pre><p>To view, visit <a href="https://review.coreboot.org/21358">change 21358</a>. To unsubscribe, visit <a href="https://review.coreboot.org/settings">settings</a>.</p><div itemscope itemtype="http://schema.org/EmailMessage"><div itemscope itemprop="action" itemtype="http://schema.org/ViewAction"><link itemprop="url" href="https://review.coreboot.org/21358"/><meta itemprop="name" content="View Change"/></div></div>

<div style="display:none"> Gerrit-Project: coreboot </div>
<div style="display:none"> Gerrit-Branch: master </div>
<div style="display:none"> Gerrit-MessageType: newchange </div>
<div style="display:none"> Gerrit-Change-Id: I2150cc14f8a0ce7f819f29c49cb00c695448bc10 </div>
<div style="display:none"> Gerrit-Change-Number: 21358 </div>
<div style="display:none"> Gerrit-PatchSet: 1 </div>
<div style="display:none"> Gerrit-Owner: Damien Zammit <damien@zamaudio.com> </div>