[all-commits] [llvm/llvm-project] 6f4f0a: [Compiler-rt] Fix bug when considering CMake path ...

Dan Liew via All-commits all-commits at lists.llvm.org
Tue Apr 20 11:58:05 PDT 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 6f4f0afaa8ae54e44e94b9358bb5336a35cb8614
      https://github.com/llvm/llvm-project/commit/6f4f0afaa8ae54e44e94b9358bb5336a35cb8614
  Author: Dan Liew <dliew at apple.com>
  Date:   2021-04-20 (Tue, 20 Apr 2021)

  Changed paths:
    M compiler-rt/cmake/Modules/CompilerRTUtils.cmake

  Log Message:
  -----------
  [Compiler-rt] Fix bug when considering CMake path returned by llvm-config.

The previous check was wrong because it only checks that the LLVM CMake
directory exists. However, it's possible that the directory exists but
the `LLVMConfig.cmake` file does not. When this happens we would
incorectly try to include the non-existant file.

To fix this we make the check stricter by checking that the file
we want to include actually exists.

This is a follow up to fd28517d878e1d3d14f492ab659aabdf729fd331.

rdar://76870467




More information about the All-commits mailing list