Hello,

As discussed in [1] and [2] the flag, preventing movt /
movw instructions is also useful outside of IOS. This
patch renames the flag in clang and depends on the
same changes in llvm.

I don't have commit privileges, so it would be nice
if someone can pick this up, preferable directly after
the llvm change, with I tend to submit hereafter (and
is actually split in two, so the llvm patch has the same
subject as the clang one (so I can be reverted if there
is a need to it...)

Regards,
Jeroen

[1] http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130805/183980.html [2] http://lists.cs.uiuc.edu/pipermail/llvm-commits/Week-of-Mon-20130812/184064.html


commit 93d5296d72dcae95d02d4561d8d96777817dce55
Author: Jeroen Hofstee <[email protected]>
Date:   Thu Aug 8 21:21:32 2013 +0200

    make arm-use-movt available for all ARM
    
    This updates clang according to a pending patch for llvm to
    rename of the -arm-darwin-use-movt to arm-use-movt to make
    it available for all of ARM.
    
    note: please apply this close to the llvm change.

diff --git lib/Driver/Tools.cpp lib/Driver/Tools.cpp
index 254bf8b..e594b16 100644
--- lib/Driver/Tools.cpp
+++ lib/Driver/Tools.cpp
@@ -821,7 +821,7 @@ void Clang::AddARMTargetArgs(const ArgList &Args,
 
     // The kext linker doesn't know how to deal with movw/movt.
     CmdArgs.push_back("-backend-option");
-    CmdArgs.push_back("-arm-darwin-use-movt=0");
+    CmdArgs.push_back("-arm-use-movt=0");
   }
 
   // Setting -mno-global-merge disables the codegen global merge pass. Setting 
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to