azharudd added a comment.

In D124474#3475432 <https://reviews.llvm.org/D124474#3475432>, @beanz wrote:

> I question whether we should be extending this or killing it off... Is there 
> a reason you're using `LLVM_BUILD_EXTERNAL_COMPILER_RT` instead of 
> `LLVM_ENABLE_RUNTIMES=compiler-rt`?

In this case I'm just trying to make it work with existing build configurations 
where it is being used (for example Swift). We should definitely be moving away 
from `LLVM_BUILD_EXTERNAL_COMPILER_RT` in favor of `LLVM_ENABLE_RUNTIMES` 
eventually though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D124474

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

Reply via email to