mirror of
https://github.com/encounter/dawn-cmake.git
synced 2025-10-10 20:09:03 +00:00
c6b381495d
Replace the ShouldRun() method with Apply() which will do the transformation if it needs to be done, otherwise returns 'SkipTransform'. This reduces a bunch of duplicated scanning between the old ShouldRun() and Transform(). This change also adjusts code style to make the transforms more consistent. Change-Id: I9a6b10cb8b4ed62676b12ef30fb7764d363386c6 Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/107681 Reviewed-by: James Price <jrprice@google.com> Kokoro: Kokoro <noreply+kokoro@google.com> Commit-Queue: Ben Clayton <bclayton@google.com> Reviewed-by: Dan Sinclair <dsinclair@chromium.org>
51 lines
1.8 KiB
C++
51 lines
1.8 KiB
C++
// Copyright 2021 The Tint Authors.
|
|
//
|
|
// Licensed under the Apache License, Version 2.0 (the "License");
|
|
// you may not use this file except in compliance with the License.
|
|
// You may obtain a copy of the License at
|
|
//
|
|
// http://www.apache.org/licenses/LICENSE-2.0
|
|
//
|
|
// Unless required by applicable law or agreed to in writing, software
|
|
// distributed under the License is distributed on an "AS IS" BASIS,
|
|
// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
|
|
// See the License for the specific language governing permissions and
|
|
// limitations under the License.
|
|
|
|
#ifndef SRC_TINT_TRANSFORM_LOCALIZE_STRUCT_ARRAY_ASSIGNMENT_H_
|
|
#define SRC_TINT_TRANSFORM_LOCALIZE_STRUCT_ARRAY_ASSIGNMENT_H_
|
|
|
|
#include "src/tint/transform/transform.h"
|
|
|
|
namespace tint::transform {
|
|
|
|
/// This transforms replaces assignment to dynamically-indexed fixed-size arrays
|
|
/// in structs on shader-local variables with code that copies the arrays to a
|
|
/// temporary local variable, assigns to the local variable, and copies the
|
|
/// array back. This is to work around FXC's compilation failure for these cases
|
|
/// (see crbug.com/tint/1206).
|
|
///
|
|
/// @note Depends on the following transforms to have been run first:
|
|
/// * SimplifyPointers
|
|
class LocalizeStructArrayAssignment final
|
|
: public Castable<LocalizeStructArrayAssignment, Transform> {
|
|
public:
|
|
/// Constructor
|
|
LocalizeStructArrayAssignment();
|
|
|
|
/// Destructor
|
|
~LocalizeStructArrayAssignment() override;
|
|
|
|
/// @copydoc Transform::Apply
|
|
ApplyResult Apply(const Program* program,
|
|
const DataMap& inputs,
|
|
DataMap& outputs) const override;
|
|
|
|
private:
|
|
struct State;
|
|
};
|
|
|
|
} // namespace tint::transform
|
|
|
|
#endif // SRC_TINT_TRANSFORM_LOCALIZE_STRUCT_ARRAY_ASSIGNMENT_H_
|