[PATCH] Fix -mthumb resetting of target cpu

Bernard Ogden bernard.ogden at arm.com
Wed Oct 30 05:34:30 PDT 2013


Hi rengolin,

Specifying -mthumb should and does change the triple from arm-foo to thumb-foo
    
In many cases where the CPU is not explicitly specified, it will also change the CPU from the target default to arm7tdmi. We fix this behaviour.
    
Adds a lot of tests. Some of these have FIXMEs associated with them. These cases did not work as expected and apply to targets that are not a priority for me right now. At least the failing cases are now flagged.

Fixes http://llvm.org/bugs/show_bug.cgi?id=17559, or at least is the first step to so doing.
   

http://llvm-reviews.chandlerc.com/D2061

Files:
  lib/Driver/ToolChain.cpp
  lib/Driver/Tools.cpp
  test/Driver/arm-cortex-cpus.c
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D2061.1.patch
Type: text/x-patch
Size: 14932 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20131030/6dc77640/attachment.bin>


More information about the cfe-commits mailing list