Don't optimize const binary subscr

Summary: Another optimization that has been removed from the AST optimizer.

Reviewed By: mpage

Differential Revision: D82128883

fbshipit-source-id: 3e94f95275ba722158142a557d3a794d63f51910

Author: DinoV

cinderx/PythonLib/cinderx/compiler/optimizer.py

@@ -509,6 +509,12 @@ def visitBinOp(self, node: ast.BinOp) -> ast.expr:
 
         return self.update_node(node, left=lhs, right=rhs)
 
+    def visitSubscript(self, node: ast.Subscript) -> ast.expr:
+        value = self.visit(node.value)
+        slice = self.visit(node.slice)
+
+        return self.update_node(node, value=value, slice=slice)
+
     def visitMatchValue(self, node: ast.MatchValue) -> ast.MatchValue:
         return self.update_node(node, value=self.fold_const_match_patterns(node.value))
 

cinderx/PythonLib/test_cinderx/test_compiler/testcorpus/02_expr_subscr_str.py

@@ -0,0 +1 @@
+x = "string"[3]