[PATCH] Fix tablegen cmake rule.

James Y Knight jyknight at google.com
Fri May 8 09:52:26 PDT 2015


Okay, right, I'll look into fixing it within the tool instead of this.


================
Comment at: cmake/modules/TableGen.cmake:31-32
@@ -30,3 +30,4 @@
     ${LLVM_TARGET_DEFINITIONS_ABSOLUTE}
-    -o ${CMAKE_CURRENT_BINARY_DIR}/${ofn}.tmp
+    -o ${CMAKE_CURRENT_BINARY_DIR}/${ofn}.tmp ||
+    { rm -f ${CMAKE_CURRENT_BINARY_DIR}/${ofn}.tmp && false \; }
     # The file in LLVM_TARGET_DEFINITIONS may be not in the current
----------------
rnk wrote:
> I would be surprised if this shell expression works on Windows. =/
Er, damnit, Windows, forgot about that.

http://reviews.llvm.org/D9608

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list