JonChesterfield added reviewers: tianshilei1992, ye-luo, RaviNarayanaswamy.
JonChesterfield added a comment.

Added some people I can remember from the original discussion.

The effect of this patch will be that clang -fopenmp foo.cpp will build an 
executable that doesn't know where its runtime libraries are. If it finds the 
right ones anyway that'll be fine. If it finds ones from some unrelated 
toolchain, that application usually won't work.

This is likely to be more painful for people with multiple toolchains installed.

How do the other projects deal with this hazard?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D143306

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

Reply via email to