Commit 5e9441a2 by Ben Clayton

Replace FIXMEs with TODOs

As requested here: https://swiftshader-review.googlesource.com/c/SwiftShader/+/31841/1#message-24da63454b89e5d545f3b686f2470c8d2392ee61 Change-Id: I76500807933b565fb13f30268c47c7caa4be7c69 Reviewed-on: https://swiftshader-review.googlesource.com/c/SwiftShader/+/31843Tested-by: 's avatarBen Clayton <bclayton@google.com> Reviewed-by: 's avatarNicolas Capens <nicolascapens@google.com> Kokoro-Presubmit: kokoro <noreply+kokoro@google.com>
parent 7b723d89
...@@ -1297,7 +1297,7 @@ namespace sw ...@@ -1297,7 +1297,7 @@ namespace sw
Int4 linear = CmpLT(c, Float4(0.04045f)); Int4 linear = CmpLT(c, Float4(0.04045f));
Float4 s = c; Float4 s = c;
s.xyz = As<Float4>((linear & As<Int4>(lc)) | (~linear & As<Int4>(ec))); // FIXME: IfThenElse() s.xyz = As<Float4>((linear & As<Int4>(lc)) | (~linear & As<Int4>(ec))); // TODO: IfThenElse()
return s; return s;
} }
......
...@@ -2195,7 +2195,7 @@ namespace sw ...@@ -2195,7 +2195,7 @@ namespace sw
default: // Wrap default: // Wrap
{ {
Int4 under = CmpLT(xyz0, Int4(0)); Int4 under = CmpLT(xyz0, Int4(0));
xyz0 = (under & maxXYZ) | (~under & xyz0); // xyz < 0 ? dim - 1 : xyz // FIXME: IfThenElse() xyz0 = (under & maxXYZ) | (~under & xyz0); // xyz < 0 ? dim - 1 : xyz // TODO: IfThenElse()
Int4 nover = CmpLT(xyz1, dim); Int4 nover = CmpLT(xyz1, dim);
xyz1 = nover & xyz1; // xyz >= dim ? 0 : xyz xyz1 = nover & xyz1; // xyz >= dim ? 0 : xyz
......
...@@ -275,7 +275,7 @@ namespace ...@@ -275,7 +275,7 @@ namespace
sw::SIMD::Int linear = CmpLT(c, sw::SIMD::Float(0.04045f)); sw::SIMD::Int linear = CmpLT(c, sw::SIMD::Float(0.04045f));
sw::SIMD::Float s = c; sw::SIMD::Float s = c;
s.xyz = rr::As<sw::SIMD::Float>((linear & rr::As<sw::SIMD::Int>(lc)) | (~linear & rr::As<sw::SIMD::Int>(ec))); // FIXME: IfThenElse() s.xyz = rr::As<sw::SIMD::Float>((linear & rr::As<sw::SIMD::Int>(lc)) | (~linear & rr::As<sw::SIMD::Int>(ec))); // TODO: IfThenElse()
return s; return s;
} }
...@@ -3486,7 +3486,7 @@ namespace sw ...@@ -3486,7 +3486,7 @@ namespace sw
for (auto i = 0u; i < type.sizeInComponents; i++) for (auto i = 0u; i < type.sizeInComponents; i++)
{ {
auto sel = cond.Int(condIsScalar ? 0 : i); auto sel = cond.Int(condIsScalar ? 0 : i);
dst.move(i, (sel & lhs.Int(i)) | (~sel & rhs.Int(i))); // FIXME: IfThenElse() dst.move(i, (sel & lhs.Int(i)) | (~sel & rhs.Int(i))); // TODO: IfThenElse()
} }
return EmitResult::Continue; return EmitResult::Continue;
......
...@@ -1174,7 +1174,7 @@ namespace sw ...@@ -1174,7 +1174,7 @@ namespace sw
Int4 linear = CmpLT(c, Float4(0.04045f)); Int4 linear = CmpLT(c, Float4(0.04045f));
Float4 s = c; Float4 s = c;
s.xyz = As<Float4>((linear & As<Int4>(lc)) | (~linear & As<Int4>(ec))); // FIXME: IfThenElse() s.xyz = As<Float4>((linear & As<Int4>(lc)) | (~linear & As<Int4>(ec))); // TODO: IfThenElse()
return s; return s;
} }
......
...@@ -2481,7 +2481,7 @@ namespace sw ...@@ -2481,7 +2481,7 @@ namespace sw
default: // Wrap default: // Wrap
{ {
Int4 under = CmpLT(xyz0, Int4(0)); Int4 under = CmpLT(xyz0, Int4(0));
xyz0 = (under & maxXYZ) | (~under & xyz0); // xyz < 0 ? dim - 1 : xyz // FIXME: IfThenElse() xyz0 = (under & maxXYZ) | (~under & xyz0); // xyz < 0 ? dim - 1 : xyz // TODO: IfThenElse()
Int4 nover = CmpLT(xyz1, dim); Int4 nover = CmpLT(xyz1, dim);
xyz1 = nover & xyz1; // xyz >= dim ? 0 : xyz xyz1 = nover & xyz1; // xyz >= dim ? 0 : xyz
......
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