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.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
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!
Comment Actions
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.