This is an archive of the discontinued LLVM Phabricator instance.

Replace string GNU Triples with llvm::Triple in MCAsmInfo subclasses and create*AsmInfo()
ClosedPublic

Authored by dsanders on Jun 4 2015, 4:01 AM.

Details

Summary

This is the first of several patches to eliminate StringRef forms of GNU
triples from the internals of LLVM. After this is complete, GNU triples
will be replaced by a more authoratitive representation in the form of
an LLVM TargetTuple.

Diff Detail

Event Timeline

dsanders updated this revision to Diff 27101.Jun 4 2015, 4:01 AM
dsanders retitled this revision from to Replace string GNU Triples with llvm::Triple in MCAsmInfo subclasses and create*AsmInfo().
dsanders updated this object.
dsanders edited the test plan for this revision. (Show Details)
dsanders added subscribers: rengolin, Unknown Object (MLST).
rengolin accepted this revision.Jun 4 2015, 5:28 AM
rengolin added a reviewer: rengolin.

No functional change, AFAICS.

The naming convention between TT, Triple and TheTriple is odd, but it's not up to this patch to change that.

LGTM. Thanks!

This revision is now accepted and ready to land.Jun 4 2015, 5:28 AM

Thanks

No functional change, AFAICS.

I forgot the NFC in the subject line. I'll include it on the commit.

The naming convention between TT, Triple and TheTriple is odd, but it's not up to this patch to change that.

I agree. The inconsistency stems from my attempt to minimize the changes. It seemed preferable to change the parameter name rather than each reference to the local constructed from it.

dsanders closed this revision.Jun 4 2015, 6:16 AM