Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(3032)

Unified Diff: src/sksl/ir/SkSLIndexExpression.h

Issue 2143323003: Revert of SkSL performance improvements (Closed) Base URL: https://skia.googlesource.com/skia@master
Patch Set: Created 4 years, 5 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « src/sksl/ir/SkSLFunctionReference.h ('k') | src/sksl/ir/SkSLIntLiteral.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: src/sksl/ir/SkSLIndexExpression.h
diff --git a/src/sksl/ir/SkSLIndexExpression.h b/src/sksl/ir/SkSLIndexExpression.h
index 78727aa5c52edf0d3aa879b3da0605517bcf4d01..538c656153ad42929bb411ef49fd85d215b6e848 100644
--- a/src/sksl/ir/SkSLIndexExpression.h
+++ b/src/sksl/ir/SkSLIndexExpression.h
@@ -16,21 +16,21 @@
/**
* Given a type, returns the type that will result from extracting an array value from it.
*/
-static const Type& index_type(const Type& type) {
+static std::shared_ptr<Type> index_type(const Type& type) {
if (type.kind() == Type::kMatrix_Kind) {
- if (type.componentType() == *kFloat_Type) {
+ if (type.componentType() == kFloat_Type) {
switch (type.columns()) {
- case 2: return *kVec2_Type;
- case 3: return *kVec3_Type;
- case 4: return *kVec4_Type;
+ case 2: return kVec2_Type;
+ case 3: return kVec3_Type;
+ case 4: return kVec4_Type;
default: ASSERT(false);
}
} else {
- ASSERT(type.componentType() == *kDouble_Type);
+ ASSERT(type.componentType() == kDouble_Type);
switch (type.columns()) {
- case 2: return *kDVec2_Type;
- case 3: return *kDVec3_Type;
- case 4: return *kDVec4_Type;
+ case 2: return kDVec2_Type;
+ case 3: return kDVec3_Type;
+ case 4: return kDVec4_Type;
default: ASSERT(false);
}
}
@@ -43,10 +43,10 @@
*/
struct IndexExpression : public Expression {
IndexExpression(std::unique_ptr<Expression> base, std::unique_ptr<Expression> index)
- : INHERITED(base->fPosition, kIndex_Kind, index_type(base->fType))
+ : INHERITED(base->fPosition, kIndex_Kind, index_type(*base->fType))
, fBase(std::move(base))
, fIndex(std::move(index)) {
- ASSERT(fIndex->fType == *kInt_Type);
+ ASSERT(fIndex->fType == kInt_Type);
}
std::string description() const override {
« no previous file with comments | « src/sksl/ir/SkSLFunctionReference.h ('k') | src/sksl/ir/SkSLIntLiteral.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698