rorth wrote:

(Some commenting the code directly didn't work for me ;-)

This check
```
!SystemTriple.isOSLinux()
```
isn't enough: it encompasses any version of Linux, not just Linux/x86_64.  
Besides, as I said, `getOrcRuntimePath` currently hardcodes the `runtimes` 
layout while one can still use the `projects` layout if one so chooses.  I'd 
rather you'd skip the test unconditionally until this is all figured out: as 
is, the cost is way too fragile and makes many unwarranted assumptions.

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

Reply via email to