diff options
| author | Fangrui Song <maskray@google.com> | 2018-11-09 19:24:48 +0000 |
|---|---|---|
| committer | Fangrui Song <maskray@google.com> | 2018-11-09 19:24:48 +0000 |
| commit | 60b7fb46e1a1548dc6dfa8c685d62fcd8ceaef77 (patch) | |
| tree | 469f7ad65d5864afee15b600295a5ac233078345 /llvm/lib/Target | |
| parent | a40cce80a7d205653a16544ed53923da3631e90a (diff) | |
| download | bcm5719-llvm-60b7fb46e1a1548dc6dfa8c685d62fcd8ceaef77.tar.gz bcm5719-llvm-60b7fb46e1a1548dc6dfa8c685d62fcd8ceaef77.zip | |
[Hexagon] Fix some -Wunused-function with LLVM_DUMP_METHOD and -Wunused-variable
llvm-svn: 346543
Diffstat (limited to 'llvm/lib/Target')
| -rw-r--r-- | llvm/lib/Target/Hexagon/HexagonFrameLowering.cpp | 2 | ||||
| -rw-r--r-- | llvm/lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp | 11 |
2 files changed, 9 insertions, 4 deletions
diff --git a/llvm/lib/Target/Hexagon/HexagonFrameLowering.cpp b/llvm/lib/Target/Hexagon/HexagonFrameLowering.cpp index 074f195b78d..f5736546a87 100644 --- a/llvm/lib/Target/Hexagon/HexagonFrameLowering.cpp +++ b/llvm/lib/Target/Hexagon/HexagonFrameLowering.cpp @@ -555,10 +555,10 @@ void HexagonFrameLowering::emitPrologue(MachineFunction &MF, bool HexagonFrameLowering::enableCalleeSaveSkip( const MachineFunction &MF) const { const auto &F = MF.getFunction(); - (void)F; // Silence unused variable in release builds assert(F.hasFnAttribute(Attribute::NoReturn) && F.getFunction().hasFnAttribute(Attribute::NoUnwind) && !F.getFunction().hasFnAttribute(Attribute::UWTable)); + (void)F; // No need to save callee saved registers if the function does not return. return MF.getSubtarget<HexagonSubtarget>().noreturnStackElim(); diff --git a/llvm/lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp b/llvm/lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp index 637fb3ace6a..b796e442d4f 100644 --- a/llvm/lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp +++ b/llvm/lib/Target/Hexagon/HexagonISelDAGToDAGHVX.cpp @@ -120,7 +120,7 @@ struct Coloring { return Color == ColorKind::Red ? ColorKind::Black : ColorKind::Red; } - void dump() const; + LLVM_DUMP_METHOD void dump() const; private: ArrayRef<Node> Order; @@ -267,7 +267,7 @@ bool Coloring::color() { return true; } -LLVM_DUMP_METHOD +#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) void Coloring::dump() const { dbgs() << "{ Order: {"; for (unsigned I = 0; I != Order.size(); ++I) { @@ -309,6 +309,7 @@ void Coloring::dump() const { dbgs() << " " << C.first << " -> " << ColorKindToName(C.second) << "\n"; dbgs() << " }\n}\n"; } +#endif namespace { // Base class of for reordering networks. They don't strictly need to be @@ -651,6 +652,7 @@ struct OpRef { IndexBits = 28, }; + LLVM_DUMP_METHOD void print(raw_ostream &OS, const SelectionDAG &G) const; private: @@ -663,7 +665,7 @@ struct NodeTemplate { MVT Ty = MVT::Other; std::vector<OpRef> Ops; - void print(raw_ostream &OS, const SelectionDAG &G) const; + LLVM_DUMP_METHOD void print(raw_ostream &OS, const SelectionDAG &G) const; }; struct ResultStack { @@ -699,10 +701,12 @@ struct ResultStack { BaseType List; + LLVM_DUMP_METHOD void print(raw_ostream &OS, const SelectionDAG &G) const; }; } // namespace +#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP) void OpRef::print(raw_ostream &OS, const SelectionDAG &G) const { if (isValue()) { OpV.getNode()->print(OS, &G); @@ -752,6 +756,7 @@ void ResultStack::print(raw_ostream &OS, const SelectionDAG &G) const { OS << '\n'; } } +#endif namespace { struct ShuffleMask { |
