[PATCH] Replace string GNU Triples with llvm::Triple in TargetMachine::getTargetTriple(). NFC.

Daniel Sanders daniel.sanders at imgtec.com
Thu Jun 11 03:27:44 PDT 2015


================
Comment at: lib/CodeGen/LLVMTargetMachine.cpp:250
@@ -248,3 +249,3 @@
 
-  Triple T(getTargetTriple());
+  Triple T(getTargetTriple().str());
   const MCSubtargetInfo &STI = *getMCSubtargetInfo();
----------------
rengolin wrote:
> No copy constructor?
It should be:
  const Triple &T = getTargetTriple();

I've cleaned this kind of thing up for lib/Target/ in http://reviews.llvm.org/D10382 but it seems I've forgotten lib/CodeGen/ in that patch and I'll need to update it.

http://reviews.llvm.org/D10381

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list