<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="http://git.reviewboard.kde.org/r/112136/">http://git.reviewboard.kde.org/r/112136/</a>

     </td>

    </tr>

   </table>

   <br />

 <pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">This review has been submitted with commit ecb29b6ca2b31ba6a7296ab325597a325806ab56 by Raphael Kubo da Costa to branch master.</pre>

 <br />

<p>- Commit</p>

<br />

<p>On August 17th, 2013, 10:31 p.m. UTC, Raphael Kubo da Costa wrote:</p>

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

 <tr>

  <td>

<div>Review request for Build System, Milian Wolff and Alexander Neundorf.</div>

<div>By Raphael Kubo da Costa.</div>

<p style="color: grey;"><i>Updated Aug. 17, 2013, 10:31 p.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;">Add a separate block for clang definitions.

Sharing compiler settings between GCC and clang does not always work: there

are flags (such as "-fno-check-new" or "-fno-reorder-blocks") that are

specific to GCC, and nothing stops these incompatibilities from becoming

bigger in the future.

Conversely, a separate clang block allows us to pass some additional flags

to clang that would have required yet another if() in the GCC block. For

now, this amounts to "-fdelayed-template-parsing".

(For KDE4, we also need -Wno-return-type-c-linkage because kdepim's

ktexteditorkabcbridge.cpp exports a function that returns a QString with C

linkage, but I hope this can be solved in a different way for kdepim5).

Last but not least, checks for bad GCC allocators or support for some flags

which are always present in clang can be avoided altogether when we know the

compiler we are using.

</pre>

  </td>

 </tr>

</table>

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

<ul style="margin-left: 3em; padding-left: 0;">

 <li>kde-modules/KDECompilerSettings.cmake <span style="color: grey">(4a9f760b971870c8963cb43bf1b13599ff39a265)</span></li>

</ul>

<p><a href="http://git.reviewboard.kde.org/r/112136/diff/" style="margin-left: 3em;">View Diff</a></p>

  </td>

 </tr>

</table>

  </div>

 </body>

</html>