<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<div class="moz-cite-prefix"><br>
+ if ( (GVar->isConstant() &&<br>
+ UnderlyingGV->isLocalLinkage(GV->getLinkage()))<br>
+ || (GVar->hasSection() &&<br>
+ StringRef(GVar->getSection()).startswith(".cp.")) )<br>
<br>
Extra spaces inside ( )?<br>
<br>
Looks good to me otherwise (assuming the clang patch is applied
first).<br>
<br>
On 09/12/13 18:03, Robert Lytton wrote:<br>
</div>
<blockquote
cite="mid:E55040AE4CA5DE4A84D2754CE295AF30014DA6A0@EXMAILBOX2.vo.spidergroup.co.uk"
type="cite">
<meta http-equiv="Content-Type" content="text/html;
charset=ISO-8859-1">
<style id="owaParaStyle" type="text/css">P {margin-top:0;margin-bottom:0;}</style>
<div style="direction: ltr;font-family: Tahoma;color:
#000000;font-size: 10pt;">Hi,<br>
<br>
Here is a patch for the xcore target<br>
<br>
XCore target: fix const section handling<br>
<br>
Xcore target ABI requires const data that is externally
visible<br>
to be handled differently if it has C-language linkage
rather than<br>
C++ language linkage.<br>
<br>
Clang now emits ".cp.rodata" section information.<br>
<br>
All other externally visible constant data will be placed in
the DP section.<br>
<br>
Robert<br>
</div>
</blockquote>
<br>
<br>
<pre class="moz-signature" cols="72">--
Richard Osborne | XMOS
<a class="moz-txt-link-freetext" href="http://www.xmos.com">http://www.xmos.com</a>
</pre>
</body>
</html>