Re: [PATCH] D17183: Make TargetInfo store an actual DataLayout instead of a string.

[Rafael Ávila de Espíndola via cfe-commits]

rafael added a subscriber: rafael.
rafael added a comment.

This is awesome!


================
Comment at: include/clang/CodeGen/BackendUtil.h:38
@@ -37,3 +37,3 @@
                          const TargetOptions &TOpts, const LangOptions &LOpts,
-                         StringRef TDesc, llvm::Module *M, BackendAction 
Action,
-                         raw_pwrite_stream *OS);
+                         llvm::DataLayout TDesc, llvm::Module *M,
+                         BackendAction Action, raw_pwrite_stream *OS);
----------------
Should TDesc be a reference?


http://reviews.llvm.org/D17183



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits