[PATCH] D49345: [CMake] Use correct variable as header install prefix

2018-07-14 Thread Petr Hosek via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL337118: [CMake] Use correct variable as header install prefix (authored by phosek, committed by ). Herald added a subscriber: llvm-commits. Changed prior to commit: https://reviews.llvm.org/D49345?vs=15

[PATCH] D49345: [CMake] Use correct variable as header install prefix

2018-07-14 Thread Petr Hosek via Phabricator via cfe-commits
phosek added a comment. Thank you! Repository: rCXX libc++ https://reviews.llvm.org/D49345 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D49345: [CMake] Use correct variable as header install prefix

2018-07-14 Thread Shoaib Meenai via Phabricator via cfe-commits
smeenai accepted this revision. smeenai added a comment. This revision is now accepted and ready to land. Seems like an obvious enough fix, so I'm jumping in to LGTM. Repository: rCXX libc++ https://reviews.llvm.org/D49345 ___ cfe-commits mailing

[PATCH] D49345: [CMake] Use correct variable as header install prefix

2018-07-14 Thread Petr Hosek via Phabricator via cfe-commits
phosek created this revision. phosek added reviewers: EricWF, mclow.lists, beanz. Herald added subscribers: cfe-commits, ldionne, christof, mgorny. This variable is already set in CMakeLists.txt but it wasn't used which means that the headers get installed into a wrong location when the per target