[PATCH] Replace string GNU Triples with llvm::Triple in MCAsmInfo subclasses and create*AsmInfo()
Daniel Sanders
daniel.sanders at imgtec.com
Thu Jun 4 06:01:29 PDT 2015
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.
http://reviews.llvm.org/D10236
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list