Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

spirv-fuzz: adds TransformationReplaceCopyObjectWithStoreLoad #3567

Merged
merged 15 commits into from
Jul 23, 2020
Merged

spirv-fuzz: adds TransformationReplaceCopyObjectWithStoreLoad #3567

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
c6b5050
Basic structure
Jul 17, 2020
004b314
Revert "Added support for floating-point comparisons"
Jul 17, 2020
1736ea7
Initial structure
Jul 19, 2020
ad9e428
Revert "Revert "Added support for floating-point comparisons""
Jul 19, 2020
65ac59b
Fixes the transformation itself.
Jul 20, 2020
ff1e3fd
Added tests.
Jul 20, 2020
7d222e4
Fixes rejection of pointer types.
Jul 21, 2020
3cd40ab
Fixes spelling in comments.
Jul 21, 2020
fe33ec6
Fixes copyrights.
Jul 21, 2020
2ba9e0a
Merge branch 'master' into t_replace_copyobject_with_store_load_3351
Jul 21, 2020
4173e6e
Merge branch 'master' into t_replace_copyobject_with_store_load_3351
Jul 22, 2020
4c083f0
Fixes according to comments.
Jul 22, 2020
7893a2d
Merge branch 't_replace_copyobject_with_store_load_3351' of https://g…
Jul 22, 2020
4b0280e
Minor fixes.
Jul 22, 2020
a39fa63
Fixes bug in recognizing pointer types.
Jul 22, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Minor fixes.
  • Loading branch information
Antoni Karpiński committed Jul 22, 2020
commit 4b0280eb12ddcae8d44bba8c50f46f79a04ce794
1 change: 1 addition & 0 deletions source/fuzz/fuzzer_pass_replace_copy_objects_with_stores_loads.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,6 +19,7 @@

namespace spvtools {
namespace fuzz {
antonikarp marked this conversation as resolved.
Show resolved Hide resolved

// Replaces instructions OpCopyObject with storing into a new variable
// and immediately loading this variable to |result_id| of the
// original OpCopyObject instruction.
Expand Down
4 changes: 2 additions & 2 deletions source/fuzz/transformation_replace_copy_object_with_store_load.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -80,8 +80,8 @@ bool TransformationReplaceCopyObjectWithStoreLoad::IsApplicable(
return false;
}
// |message_.variable_storage_class| must be Private or Function.
return (message_.variable_storage_class() == SpvStorageClassPrivate ||
message_.variable_storage_class() == SpvStorageClassFunction);
return message_.variable_storage_class() == SpvStorageClassPrivate ||
message_.variable_storage_class() == SpvStorageClassFunction;
}

void TransformationReplaceCopyObjectWithStoreLoad::Apply(
Expand Down