[PATCH] D16513: [Unittest] Clean up formatting, NFC
David Blaikie via llvm-commits
llvm-commits at lists.llvm.org
Thu Jan 28 22:08:56 PST 2016
dblaikie added inline comments.
================
Comment at: unittests/ExecutionEngine/Orc/ObjectTransformLayerTest.cpp:327
@@ +326,3 @@
+ std::vector<llvm::Module *> Modules;
+ CompileLayer.addModuleSet(Modules, &Manager, &Resolver);
+
----------------
I think I mentioned this during post-commit review, perhaps.
It's probably possible to just write "{}" for the first parameter - skipping both the local variable and the inline construction. (though, FWIW, I'd probably take the inline construction over the named variable if those were the choices - but I can see the alternative point of view/wouldn't say it's clear cut either way)
http://reviews.llvm.org/D16513
More information about the llvm-commits
mailing list