XCore target: fix weak linkage attribute handling

Richard Osborne richard at xmos.com
Fri Aug 16 09:07:40 PDT 2013


Looks good to me.


--- a/lib/Target/XCore/MCTargetDesc/XCoreMCAsmInfo.cpp
+++ b/lib/Target/XCore/MCTargetDesc/XCoreMCAsmInfo.cpp
@@ -23,7 +23,6 @@ XCoreMCAsmInfo::XCoreMCAsmInfo(StringRef TT) {

    PrivateGlobalPrefix = ".L";
    AscizDirective = ".asciiz";
-  WeakDefDirective = "\t.weak\t";
    WeakRefDirective = "\t.weak\t";

Should we also remove the override of the WeakRefDirective, or is this 
still needed?

On 16/08/13 16:57, Robert Lytton wrote:
> Hi,
>
> Here is a patch to remove WeakDefDirective as there is no directive.
>
> Robert
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits


-- 
Richard Osborne | XMOS
http://www.xmos.com

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130816/1966c91e/attachment.html>


More information about the llvm-commits mailing list