<html>
 <body>
  <div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
   <table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
    <tr>
     <td>
      This is an automatically generated e-mail. To reply, visit:
      <a href="https://reviewboard.asterisk.org/r/1534/">https://reviewboard.asterisk.org/r/1534/</a>
     </td>
    </tr>
   </table>
   <br />





<blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
 <p style="margin-top: 0;">On November 6th, 2011, 5:20 a.m., <b>Tzafrir Cohen</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Sorry. I must have missed it. This whole thing should go away. Those tests may belong in the configure script, if at all. You should also note that the build system is not necessarily the running system.

Anyway, what is the minimal version of GCC supported by Asterisk nowadays?</pre>
 </blockquote>




 <p>On November 7th, 2011, 3:02 a.m., <b>Tzafrir Cohen</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">When I asked about it on a certain sparc IRC channel someone noted that setting -mcpu is normally unnecessary.</pre>
 </blockquote>





 <p>On November 8th, 2011, 5:17 a.m., <b>wdoekes</b> wrote:</p>
 <blockquote style="margin-left: 1em; border-left: 2px solid #d0d0d0; padding-left: 10px;">
  <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Ok. Tzafrir and I agree that the whole mcpu setting should not be done in the Makefile at all.

Something like this should probably be a start:


Index: Makefile
===================================================================
--- Makefile        (revision 343785)
+++ Makefile        (working copy)
@@ -169,14 +169,11 @@
   endif
 
   ifeq ($(PROC),sparc64)
-    #The problem with sparc is the best stuff is in newer versions of gcc (post 3.0) only.
-    #This works for even old (2.96) versions of gcc and provides a small boost either way.
-    #A ultrasparc cpu is really v9 but the stock debian stable 3.0 gcc doesn&#39;t support it.
-    #So we go lowest common available by gcc and go a step down, still a step up from
-    #the default as we now have a better instruction set to work with. - Belgarath
+    # This conditional CC stuff should go away. If you&#39;re cross compiling, things will break.
+    # If you need special CFLAGS, provide them in the environment before calling
+    # ./configure.
     PROC=ultrasparc
     OPTIONS+=$(shell if $(CC) -mtune=$(PROC) -S -o /dev/null -xc /dev/null &gt;/dev/null 2&gt;&amp;1; then echo &quot;-mtune=$(PROC)&quot;; fi)
-    OPTIONS+=$(shell if $(CC) -mcpu=v8 -S -o /dev/null -xc /dev/null &gt;/dev/null 2&gt;&amp;1; then echo &quot;-mcpu=v8&quot;; fi)
     OPTIONS+=-fomit-frame-pointer
   endif
 
@@ -221,9 +218,7 @@
 endif
 
 ifeq ($(findstring -march,$(_ASTCFLAGS) $(ASTCFLAGS)),)
-  ifneq ($(PROC),ultrasparc)
-    _ASTCFLAGS+=$(shell if $(CC) -march=$(PROC) -S -o /dev/null -xc /dev/null &gt;/dev/null 2&gt;&amp;1; then echo &quot;-march=$(PROC)&quot;; fi)
-  endif
+  _ASTCFLAGS+=$(shell if $(CC) -march=$(PROC) -S -o /dev/null -xc /dev/null &gt;/dev/null 2&gt;&amp;1; then echo &quot;-march=$(PROC)&quot;; fi)
 endif
 
 ifeq ($(PROC),ppc)
</pre>
 </blockquote>








</blockquote>

<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">That whole piece of code is pointless. If anybody tries to build Asterisk on a Woody system with gcc 3.0, we&#39;ll point them to the workaround (if other stuff actually works). But otherwise, that whole part should just go. I have a patch in the Debian package to remove it.</pre>
<br />








<p>- Tzafrir</p>


<br />
<p>On October 20th, 2011, 12:56 a.m., Alec Davis wrote:</p>






<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('https://reviewboard.asterisk.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
 <tr>
  <td>

<div>Review request for Asterisk Developers.</div>
<div>By Alec Davis.</div>


<p style="color: grey;"><i>Updated Oct. 20, 2011, 12:56 a.m.</i></p>




<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Debian Squeeze or Ubuntu 9.04.

/usr/local/src/asterisk-1.8.7.0/include/asterisk/lock.h:600: undefined reference to `__sync_fetch_and_add_4&#39;

Previous Debian releases are fine.</pre>
  </td>
 </tr>
</table>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Testing </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
 <tr>
  <td>
   <pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Tested sucessfully with;
Debian Squeeze - where it would previously emit the above message and more.
Debian Lenny - where it used to work without mcpu=v9
Debian Etch - where it used to work without mcpu=v9
</pre>
  </td>
 </tr>
</table>



<div style="margin-top: 1.5em;">
 <b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>


 <a href="https://issues.asterisk.org/jira/browse/ASTERISK-17022">ASTERISK-17022</a>, 

 <a href="https://issues.asterisk.org/jira/browse/ASTERISK-18651">ASTERISK-18651</a>


</div>


<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">

 <li>trunk/Makefile <span style="color: grey">(341525)</span></li>

</ul>

<p><a href="https://reviewboard.asterisk.org/r/1534/diff/" style="margin-left: 3em;">View Diff</a></p>




  </td>
 </tr>
</table>








  </div>
 </body>
</html>