[PATCH] D67904: [ARM][MVE] validForTailPredication insts

Danila Malyutin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 26 05:30:12 PDT 2019


danilaml added inline comments.


================
Comment at: unittests/Target/ARM/CMakeLists.txt:2
 include_directories(
-  ${CMAKE_SOURCE_DIR}/lib/Target/ARM
-  ${CMAKE_BINARY_DIR}/lib/Target/ARM
+  ${LLVM_MAIN_SRC_DIR}/lib/Target/ARM
+  ${LLVM_BINARY_DIR}/lib/Target/ARM
----------------
Thanks! I realize that two other tests in this (Target) folder also have the same pattern in their CMakeLists and would benefit from the same adjustment. Not sure if it makes sense to include them in the same patch though.


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

https://reviews.llvm.org/D67904





More information about the llvm-commits mailing list