Skip to content

Commit 24980f1

Browse files
committed
add module.__getattribute__ which falls back to locally defined getattr
1 parent 04e4556 commit 24980f1

File tree

3 files changed

+104
-0
lines changed

3 files changed

+104
-0
lines changed

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/Python3Core.java

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -135,6 +135,7 @@
135135
import com.oracle.graal.python.builtins.objects.method.MethodBuiltins;
136136
import com.oracle.graal.python.builtins.objects.method.StaticmethodBuiltins;
137137
import com.oracle.graal.python.builtins.objects.mmap.MMapBuiltins;
138+
import com.oracle.graal.python.builtins.objects.module.ModuleBuiltins;
138139
import com.oracle.graal.python.builtins.objects.module.PythonModule;
139140
import com.oracle.graal.python.builtins.objects.object.ObjectBuiltins;
140141
import com.oracle.graal.python.builtins.objects.object.PythonObject;
@@ -293,6 +294,7 @@ private static final PythonBuiltins[] initializeBuiltins() {
293294
new ArrayModuleBuiltins(),
294295
new ArrayBuiltins(),
295296
new TimeModuleBuiltins(),
297+
new ModuleBuiltins(),
296298
new MathModuleBuiltins(),
297299
new MarshalModuleBuiltins(),
298300
new RandomModuleBuiltins(),
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,97 @@
1+
/*
2+
* Copyright (c) 2019, Oracle and/or its affiliates. All rights reserved.
3+
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
4+
*
5+
* The Universal Permissive License (UPL), Version 1.0
6+
*
7+
* Subject to the condition set forth below, permission is hereby granted to any
8+
* person obtaining a copy of this software, associated documentation and/or
9+
* data (collectively the "Software"), free of charge and under any and all
10+
* copyright rights in the Software, and any and all patent rights owned or
11+
* freely licensable by each licensor hereunder covering either (i) the
12+
* unmodified Software as contributed to or provided by such licensor, or (ii)
13+
* the Larger Works (as defined below), to deal in both
14+
*
15+
* (a) the Software, and
16+
*
17+
* (b) any piece of software and/or hardware listed in the lrgrwrks.txt file if
18+
* one is included with the Software each a "Larger Work" to which the Software
19+
* is contributed by such licensors),
20+
*
21+
* without restriction, including without limitation the rights to copy, create
22+
* derivative works of, display, perform, and distribute the Software and make,
23+
* use, sell, offer for sale, import, export, have made, and have sold the
24+
* Software and the Larger Work(s), and to sublicense the foregoing rights on
25+
* either these or other terms.
26+
*
27+
* This license is subject to the following condition:
28+
*
29+
* The above copyright notice and either this complete permission notice or at a
30+
* minimum a reference to the UPL must be included in all copies or substantial
31+
* portions of the Software.
32+
*
33+
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
34+
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
35+
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
36+
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
37+
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
38+
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
39+
* SOFTWARE.
40+
*/
41+
package com.oracle.graal.python.builtins.objects.module;
42+
43+
import static com.oracle.graal.python.nodes.SpecialMethodNames.__GETATTRIBUTE__;
44+
import static com.oracle.graal.python.nodes.SpecialMethodNames.__GETATTR__;
45+
46+
import java.util.List;
47+
48+
import com.oracle.graal.python.builtins.Builtin;
49+
import com.oracle.graal.python.builtins.CoreFunctions;
50+
import com.oracle.graal.python.builtins.PythonBuiltinClassType;
51+
import com.oracle.graal.python.builtins.PythonBuiltins;
52+
import com.oracle.graal.python.builtins.objects.PNone;
53+
import com.oracle.graal.python.builtins.objects.object.ObjectBuiltins;
54+
import com.oracle.graal.python.nodes.attributes.ReadAttributeFromObjectNode;
55+
import com.oracle.graal.python.nodes.call.CallNode;
56+
import com.oracle.graal.python.nodes.function.PythonBuiltinBaseNode;
57+
import com.oracle.graal.python.nodes.function.builtins.PythonBinaryBuiltinNode;
58+
import com.oracle.graal.python.nodes.object.IsBuiltinClassProfile;
59+
import com.oracle.graal.python.runtime.exception.PException;
60+
import com.oracle.truffle.api.dsl.Cached;
61+
import com.oracle.truffle.api.dsl.GenerateNodeFactory;
62+
import com.oracle.truffle.api.dsl.NodeFactory;
63+
import com.oracle.truffle.api.dsl.Specialization;
64+
import com.oracle.truffle.api.profiles.ConditionProfile;
65+
66+
@CoreFunctions(extendClasses = PythonBuiltinClassType.PythonModule)
67+
public class ModuleBuiltins extends PythonBuiltins {
68+
69+
@Override
70+
protected List<? extends NodeFactory<? extends PythonBuiltinBaseNode>> getNodeFactories() {
71+
return ModuleBuiltinsFactory.getFactories();
72+
}
73+
74+
@Builtin(name = __GETATTRIBUTE__, minNumOfPositionalArgs = 2)
75+
@GenerateNodeFactory
76+
public abstract static class ModuleGetattritbuteNode extends PythonBinaryBuiltinNode {
77+
@Specialization
78+
public Object getattribute(PythonModule self, Object key,
79+
@Cached("create()") IsBuiltinClassProfile isAttrError,
80+
@Cached("create()") ObjectBuiltins.GetAttributeNode objectGetattrNode,
81+
@Cached("create()") ReadAttributeFromObjectNode readGetattr,
82+
@Cached("createBinaryProfile()") ConditionProfile customGetAttr,
83+
@Cached("create()") CallNode callNode) {
84+
try {
85+
return objectGetattrNode.execute(self, key);
86+
} catch (PException e) {
87+
e.expect(PythonBuiltinClassType.AttributeError, isAttrError);
88+
Object getAttr = readGetattr.execute(self, __GETATTR__);
89+
if (customGetAttr.profile(getAttr != PNone.NO_VALUE)) {
90+
return callNode.execute(null, getAttr, key);
91+
} else {
92+
throw e;
93+
}
94+
}
95+
}
96+
}
97+
}

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/object/ObjectBuiltins.java

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -65,6 +65,7 @@
6565
import com.oracle.graal.python.builtins.objects.dict.PDict;
6666
import com.oracle.graal.python.builtins.objects.function.PBuiltinFunction;
6767
import com.oracle.graal.python.builtins.objects.function.PKeyword;
68+
import com.oracle.graal.python.builtins.objects.object.ObjectBuiltinsFactory.GetAttributeNodeFactory;
6869
import com.oracle.graal.python.builtins.objects.type.PythonAbstractClass;
6970
import com.oracle.graal.python.builtins.objects.type.LazyPythonClass;
7071
import com.oracle.graal.python.builtins.objects.type.PythonBuiltinClass;
@@ -466,6 +467,10 @@ private LazyPythonClass getDataClass(Object descr) {
466467
}
467468
return getDataClassNode.execute(descr);
468469
}
470+
471+
public static GetAttributeNode create() {
472+
return GetAttributeNodeFactory.create();
473+
}
469474
}
470475

471476
@Builtin(name = __GETATTR__, minNumOfPositionalArgs = 2)

0 commit comments

Comments
 (0)