krytarowski abandoned this revision.
krytarowski added a comment.
Will be addressed in LLVM/Clang.
Repository:
rL LLVM
https://reviews.llvm.org/D22999
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman
labath requested changes to this revision.
labath added a comment.
This revision now requires changes to proceed.
I agree with Tamas that this should somehow be handled in a different way. llvm
already installs a bunch of cmake files. I think we should figure out a way to
get the things we need
krytarowski added a comment.
ping
Repository:
rL LLVM
https://reviews.llvm.org/D22999
___
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
krytarowski added a comment.
In https://reviews.llvm.org/D22999#501999, @tberghammer wrote:
> I think it is a bad idea to duplicate the CheckAtomic.cmake file from LLVM as
> the 2 copy will diverge over time and cause extra maintenance cost. What do
> you think about trying to change LLVM to in
tberghammer added a subscriber: tberghammer.
tberghammer added a comment.
I think it is a bad idea to duplicate the CheckAtomic.cmake file from LLVM as
the 2 copy will diverge over time and cause extra maintenance cost. What do you
think about trying to change LLVM to install this file instead o
krytarowski created this revision.
krytarowski added reviewers: labath, zturner.
krytarowski added subscribers: lldb-commits, llvm-commits, joerg.
krytarowski set the repository for this revision to rL LLVM.
LLVM doesn't install "CheckAtomic" cmake file, so add it locally.
Bump required CMake vers
