sbc100 added a comment.

Kinds seems like two different changes here.   But lgtm.



================
Comment at: clang/lib/Driver/ToolChains/WebAssembly.cpp:137
+      getFilePaths().push_back(getDriver().SysRoot + "/lib/" + MultiarchTriple 
+
+                               "/llvm-lto");
+    }
----------------
Is there any kind of president here?   i.e. do any other platforms have support 
for this kind thing?  Seems like good idea I'd just like to be sure we follow 
existing practices.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D70500/new/

https://reviews.llvm.org/D70500



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

Reply via email to