<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>