rjmccall added inline comments.
================
Comment at: lib/Driver/ToolChains/Hurd.cpp:84
+  const Driver &D = getDriver();
+  std::string SysRoot = computeSysRoot();
+
----------------
kristina wrote:
> Move semantics? Or is this guaranteed elision (I'm not sure)?
If you're talking about the initialization of `SysRoot`, yes, initializing a 
local variable from a temporary is a guaranteed copy-elision, and adding 
`std::move` will actually inhibit the optimization.


Repository:
  rC Clang

https://reviews.llvm.org/D54379



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

Reply via email to