Commit 84d0f7cb by Antonio Maiorano

Revert "Fix materialization of function arguments"

This reverts commit 51d98676. No longer needed, as the root problem was fixed in 7fefd483 (https://swiftshader-review.googlesource.com/c/SwiftShader/+/37273) Bug: b/129757459 Change-Id: I70da0bf00fd737e395a264c1f8bfd4b25325e4bc Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/37930 Kokoro-Presubmit: kokoro <noreply+kokoro@google.com> Reviewed-by: 's avatarNicolas Capens <nicolascapens@google.com> Tested-by: 's avatarAntonio Maiorano <amaiorano@google.com>
parent f523485e
...@@ -185,7 +185,6 @@ namespace rr ...@@ -185,7 +185,6 @@ namespace rr
Bool::Bool(Argument<Bool> argument) Bool::Bool(Argument<Bool> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -261,7 +260,6 @@ namespace rr ...@@ -261,7 +260,6 @@ namespace rr
Byte::Byte(Argument<Byte> argument) Byte::Byte(Argument<Byte> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -519,7 +517,6 @@ namespace rr ...@@ -519,7 +517,6 @@ namespace rr
SByte::SByte(Argument<SByte> argument) SByte::SByte(Argument<SByte> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -765,7 +762,6 @@ namespace rr ...@@ -765,7 +762,6 @@ namespace rr
Short::Short(Argument<Short> argument) Short::Short(Argument<Short> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -1004,7 +1000,6 @@ namespace rr ...@@ -1004,7 +1000,6 @@ namespace rr
UShort::UShort(Argument<UShort> argument) UShort::UShort(Argument<UShort> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -2220,7 +2215,6 @@ namespace rr ...@@ -2220,7 +2215,6 @@ namespace rr
Int::Int(Argument<Int> argument) Int::Int(Argument<Int> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -2660,7 +2654,6 @@ namespace rr ...@@ -2660,7 +2654,6 @@ namespace rr
UInt::UInt(Argument<UInt> argument) UInt::UInt(Argument<UInt> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
...@@ -3884,7 +3877,6 @@ namespace rr ...@@ -3884,7 +3877,6 @@ namespace rr
Float::Float(Argument<Float> argument) Float::Float(Argument<Float> argument)
{ {
materialize(); // FIXME(b/129757459)
storeValue(argument.value); storeValue(argument.value);
} }
......
...@@ -2888,7 +2888,6 @@ namespace rr ...@@ -2888,7 +2888,6 @@ namespace rr
template<class T> template<class T>
Pointer<T>::Pointer(Argument<Pointer<T>> argument) : alignment(1) Pointer<T>::Pointer(Argument<Pointer<T>> argument) : alignment(1)
{ {
LValue<Pointer<T>>::materialize(); // FIXME(b/129757459)
LValue<Pointer<T>>::storeValue(argument.value); LValue<Pointer<T>>::storeValue(argument.value);
} }
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment