Remove duplicate `Pass` suffix from ViewOpGraph class name

[bondhugula]

Summary

Remove duplicate Pass suffix from view-op-graph pass class name. The
extra suffix would lead to methods like registerViewOpGraphPassPass
being generated.

[jpienaar]

This is an inconsistency that we need to fix going the opposite way: ops have Op as suffix, types have Type ad suffix, etc but passes we didn't do that. We are in inconsistent state at the moment, so this is fine until we swing all (I had an initial change for that but never landed it as Mehdi wanted more changes along with first change and I never got back to it)

mlir/include/mlir/Transforms/Passes.td

let description = [{
     ```
 
     See [Symbols and SymbolTables](SymbolsAndSymbolTables.md) for more
     information on `Symbols`.
   }];
   let constructor = "mlir::createSymbolDCEPass()";
 }
 
-def ViewOpGraphPass : Pass<"view-op-graph"> {
+def ViewOpGraph : Pass<"view-op-graph"> {
   let summary = "Print Graphviz visualization of an operation";
   let description = [{
     This pass prints a Graphviz graph of a module.
 
     - Operations are represented as nodes;
     - Uses (data flow) as edges;
     - Control flow as dashed edges;
     - Regions/blocks as subgraphs.

mlir/lib/Transforms/ViewOpGraph.cpp

public:
 
   int id;
   Optional<int> clusterId;
 };
 
 /// This pass generates a Graphviz dataflow visualization of an MLIR operation.
 /// Note: See https://www.graphviz.org/doc/info/lang.html for more information
 /// about the Graphviz DOT language.
-class PrintOpPass : public ViewOpGraphPassBase<PrintOpPass> {
+class PrintOpPass : public ViewOpGraphBase<PrintOpPass> {
 public:
   PrintOpPass(raw_ostream &os) : os(os) {}
   PrintOpPass(const PrintOpPass &o) : PrintOpPass(o.os.getOStream()) {}
 
   void runOnOperation() override {
     emitGraph([&]() {
       processOperation(getOperation());
       emitAllEdgeStmts();