[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