[PATCH] Replace string GNU Triples with llvm::Triple in InitMCObjectFileInfo. NFC.

Daniel Sanders daniel.sanders at imgtec.com
Fri Jun 12 06:39:21 PDT 2015


Fixed the two redundant constructor calls.
The third is already fixed by a subsequent patch.
The remaining comment would hit the argument shadowing field problem I ran into the other day.

There wasn't a conditional LGTM so I ought to double-check: Ok to commit?


http://reviews.llvm.org/D10366

Files:
  include/llvm/MC/MCObjectFileInfo.h
  lib/MC/MCObjectFileInfo.cpp
  lib/Object/IRObjectFile.cpp
  lib/Target/TargetLoweringObjectFile.cpp
  tools/dsymutil/DwarfLinker.cpp
  tools/llvm-mc/llvm-mc.cpp

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D10366.27576.patch
Type: text/x-patch
Size: 5478 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150612/4cc9cba9/attachment.bin>


More information about the llvm-commits mailing list