[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