rorth wrote:

The new check is certainly better since it explicitly excludes anything but 
macOS and Linux/**x86_64** which are the only ones that are supported right 
now.  However, the code still doesn't account for the two possible layouts of 
the runtime dir.  Rather than hardcoding those things, it would be way better 
to just determine them at runtime just like `clang --print-runtime-dir` does.

https://github.com/llvm/llvm-project/pull/152562
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to