From a0022964276a538e37bb5bafb14136fd0d17a9b3 Mon Sep 17 00:00:00 2001 From: Sid Manning Date: Wed, 15 Oct 2014 20:41:17 +0000 Subject: [PATCH] Wrong attribute. LLVM_ATTRIBUTE_UNUSED not LLVM_ATTRIBUTE_USED This original fix for the build break was correct. LLVM_ATTRIBUTE_USED removes the warning message because it keeps the function in the object file. LLVM_ATTRIBUTE_UNUSED indicates that it may or may not be used depending on build settings. llvm-svn: 219846 --- llvm/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llvm/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp b/llvm/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp index 9816d5b..a526942 100644 --- a/llvm/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp +++ b/llvm/lib/Target/Hexagon/InstPrinter/HexagonInstPrinter.cpp @@ -63,7 +63,7 @@ static bool isExtended(uint64_t TSFlags) { } // Currently just used in an assert statement -static bool isExtendable(uint64_t TSFlags) LLVM_ATTRIBUTE_USED; +static bool isExtendable(uint64_t TSFlags) LLVM_ATTRIBUTE_UNUSED; // Return true if the instruction may be extended based on the operand value. static bool isExtendable(uint64_t TSFlags) { return (TSFlags >> HexagonII::ExtendablePos) & HexagonII::ExtendableMask; -- 2.7.4