Skip to content
Snippets Groups Projects
  • Daniel Sanders's avatar
    a6cfb95d
    Re-commit r247683: Replace Triple with a new TargetTuple in MCTargetDesc/* and related. NFC. · a6cfb95d
    Daniel Sanders authored
    Summary:
    This is the first patch in the series to migrate Triple's (which are ambiguous)
    to TargetTuple's (which aren't).
    
    For the moment, TargetTuple simply passes all requests to the Triple object it
    holds. Once it has replaced Triple, it will start to implement the interface in
    a more suitable way.
    
    This change makes some changes to the public C++ API. In particular,
    InitMCSubtargetInfo(), createMCRelocationInfo(), and createMCSymbolizer()
    now take TargetTuples instead of Triples. The other public C++ API's have
    been left as-is for the moment to reduce patch size.
    
    This commit also contains a trivial patch to clang to account for the C++ API
    change. Thanks go to Pavel Labath for fixing LLDB for me.
    
    Reviewers: rengolin
    
    Subscribers: jyknight, dschuff, arsenm, rampitec, danalbert, srhines, javed.absar, dsanders, echristo, emaste, jholewinski, tberghammer, ted, jfb, llvm-commits, rengolin
    
    Differential Revision: http://reviews.llvm.org/D10969
    
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@247692 91177308-0d34-0410-b5e6-96231b3b80d8
    a6cfb95d
    History
    Re-commit r247683: Replace Triple with a new TargetTuple in MCTargetDesc/* and related. NFC.
    Daniel Sanders authored
    Summary:
    This is the first patch in the series to migrate Triple's (which are ambiguous)
    to TargetTuple's (which aren't).
    
    For the moment, TargetTuple simply passes all requests to the Triple object it
    holds. Once it has replaced Triple, it will start to implement the interface in
    a more suitable way.
    
    This change makes some changes to the public C++ API. In particular,
    InitMCSubtargetInfo(), createMCRelocationInfo(), and createMCSymbolizer()
    now take TargetTuples instead of Triples. The other public C++ API's have
    been left as-is for the moment to reduce patch size.
    
    This commit also contains a trivial patch to clang to account for the C++ API
    change. Thanks go to Pavel Labath for fixing LLDB for me.
    
    Reviewers: rengolin
    
    Subscribers: jyknight, dschuff, arsenm, rampitec, danalbert, srhines, javed.absar, dsanders, echristo, emaste, jholewinski, tberghammer, ted, jfb, llvm-commits, rengolin
    
    Differential Revision: http://reviews.llvm.org/D10969
    
    
    git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@247692 91177308-0d34-0410-b5e6-96231b3b80d8
Code owners
Assign users and groups as approvers for specific file changes. Learn more.