[PATCH] D39218: [WebAssembly] Include libclang_rt.builtins in the standard way

Derek Schuff via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Oct 23 17:41:19 PDT 2017


dschuff added inline comments.


================
Comment at: lib/Driver/ToolChain.cpp:318
+  else
+    OSLibName = getOS();
   llvm::sys::path::append(Path, "lib", OSLibName);
----------------
Is this logic intended to replace what was removed from CommonArgs.cpp? Should there be an assert here too?


https://reviews.llvm.org/D39218





More information about the cfe-commits mailing list