!fixup address comments, thanks!

Author: fhahn

llvm/lib/Transforms/Vectorize/VPlan.h

@@ -966,11 +966,10 @@ class VPIRMetadata {
   /// nodes that are common to both.
   void intersect(const VPIRMetadata &MD);
 
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
   /// Print metadata with node IDs.
   void print(raw_ostream &O, const Module *M) const;
-
-  /// Return true if there is any metadata to print.
-  bool empty() const { return Metadata.empty(); }
+#endif
 };
 
 /// This is a concrete Recipe that models a single VPlan-level instruction.

llvm/lib/Transforms/Vectorize/VPlanRecipes.cpp

@@ -1676,6 +1676,7 @@ void VPIRMetadata::intersect(const VPIRMetadata &Other) {
   Metadata = std::move(MetadataIntersection);
 }
 
+#if !defined(NDEBUG) || defined(LLVM_ENABLE_DUMP)
 void VPIRMetadata::print(raw_ostream &O, const Module *M) const {
   if (Metadata.empty())
     return;
@@ -1694,6 +1695,7 @@ void VPIRMetadata::print(raw_ostream &O, const Module *M) const {
   });
   O << ")";
 }
+#endif
 
 void VPWidenCallRecipe::execute(VPTransformState &State) {
   assert(State.VF.isVector() && "not widening");