Use shared getScope instead of getRoot

Author: owen-mc

go/ql/lib/semmle/go/controlflow/BasicBlocks.qll

@@ -48,9 +48,6 @@ private module BbImpl = BB::Make<Location, Input>;
 class BasicBlock extends BbImpl::BasicBlock {
   /** Gets the basic block that immediately dominates this basic block. */
   ReachableBasicBlock getImmediateDominator() { result.immediatelyDominates(this) }
-
-  /** Gets the innermost function or file to which this basic block belongs. */
-  ControlFlow::Root getRoot() { result = this.getScope() }
 }
 
 cached

go/ql/lib/semmle/go/dataflow/SSA.qll

@@ -144,7 +144,7 @@ class SsaDefinition extends TSsaDefinition {
   abstract string prettyPrintRef();
 
   /** Gets the innermost function or file to which this SSA definition belongs. */
-  ControlFlow::Root getRoot() { result = this.getBasicBlock().getRoot() }
+  ControlFlow::Root getRoot() { result = this.getBasicBlock().getScope() }
 
   /** Gets a textual representation of this element. */
   string toString() { result = this.prettyPrintDef() }

go/ql/lib/semmle/go/dataflow/SsaImpl.qll

@@ -86,15 +86,15 @@ private module Internal {
 
   /** Holds if the `i`th node of `bb` in function `f` is an entry node. */
   private predicate entryNode(FuncDef f, ReachableBasicBlock bb, int i) {
-    f = bb.getRoot() and
+    f = bb.getScope() and
     bb.getNode(i).isEntryNode()
   }
 
   /**
    * Holds if the `i`th node of `bb` in function `f` is a function call.
    */
   private predicate callNode(FuncDef f, ReachableBasicBlock bb, int i) {
-    f = bb.getRoot() and
+    f = bb.getScope() and
     bb.getNode(i).(IR::EvalInstruction).getExpr() instanceof CallExpr
   }