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.

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

Refactor dup'd arg handling functions into a single location #660

Merged
merged 3 commits into from Jun 7, 2019

Conversation

zoddicus
Copy link
Collaborator

@zoddicus zoddicus commented Jun 5, 2019

Extract functions from glslc & spvc into libshaderc_util

Fixes #652

Extract functions from glslc & spvc into libshaderc_util

Fuxes google#652
@zoddicus zoddicus self-assigned this Jun 5, 2019
@zoddicus zoddicus merged commit 8738101 into google:master Jun 7, 2019
@zoddicus zoddicus deleted the issue652 branch June 7, 2019 18:24
dneto0 added a commit to dneto0/shaderc that referenced this pull request Nov 14, 2019
Including:
ef2b7a6 Avoid C-style cast for constants definitions (google#760)
c1af9a7 SetBeforeHlslLegalization to use more relaxed validation rules (google#676)
72988e4 Explicitly use python3 for git-sync-deps (google#759)
1f2dec1 Remove app_dummy call in Android test (google#756)
e9bb8f2 Add verbose flag to run_spirv_cross_tests.py (google#753)
f396afa Rolling 4 dependencies and update known_failures (google#755)
77d7b65 Rolling 4 dependencies and updating known_failures (google#754)
23bbb32 Add --target-spv option to set target SPIR-V version (google#750)
8de4816 Remove old logging infra from run_spirv_cross_tests.py (google#749)
30d8262 Enable implicit fallthrough warning and fix examples of (google#747)
7781794 Rolling 2 dependencies (google#748)
48e07b5 Rolling 4 dependencies and update known_failures (google#745)
cff2a56 Catch low level exceptions instead of crash (google#744)
11cf0a3 Add jobs flag to spirv cross tests script (google#743)
0ce3725 Extract common test environment bits out into a class (google#738)
c4ba097 Fix BUILD.gn for Chromium build. (google#741)
ab08017 BUILD.gn fixes for Fuchsia. (google#733)
b9a1021 Remove Appveyor artifacts link. (google#739)
16f6e6a Remove SetShaderModel from API (google#734)
b3523d5 Move script fixes from other SPIRV related repos (google#732)
a731ee3 Import improvements to DEPS roll script (google#729)
213a2a8 Update CHANGES (google#730)
2cbf790 Add -fnan-clamp (google#725)
6c4c6ed Add script to generate manual roll patch (google#727)
4714ba1 Rolling 4 dependencies (google#728)
2a888de Roll third_party/glslang def9662348b0..9db72785beb3 (1 commits) (google#721)
daf804a Roll third_party/spirv-cross 4104e363005a..146dc453bcec (2 commits) (google#722)
9164aea Manually roll DEPS to get past test failure (google#717)
8b2600f Roll third_party/googletest fd20d1eccef6..176eccfb8f42 (1 commits) (google#718)
f40bcf8 Roll third_party/googletest e110929a7b49..fd20d1eccef6 (5 commits) (google#715)
8fbd7e0 Roll third_party/re2 0c95bcce2f1f..848dfb7e1d7b (2 commits) (google#712)
a0bfa18 Roll third_party/googletest f7c178ecb33c..e110929a7b49 (2 commits) (google#709)
7b84ab7 Roll third_party/googletest 26afdba792e5..f7c178ecb33c (2 commits) (google#707)
828e3a5 Roll third_party/spirv-cross 00a8539d1ddf..4104e363005a (3 commits) (google#698)
9745f0e Roll third_party/googletest af4c2cb098a3..26afdba792e5 (1 commits) (google#703)
c21dd10 Update glslc tests for -finvert-y option (google#704)
ceff3dd Pass InvertY to glslang (google#695)
b988e6b Roll third_party/googletest 89656ddbe62f..af4c2cb098a3 (2 commits) (google#702)
c6cb20d Roll third_party/spirv-headers 9cf7c3a7d2d2..de99d4d834ae (1 commits) (google#697)
bd60de4 Roll third_party/googletest 076b7f778883..89656ddbe62f (1 commits) (google#694)
dcfa9d0 Roll third_party/spirv-cross fccf1d046204..00a8539d1ddf (1 commits) (google#691)
ef6b960 Roll third_party/spirv-cross 5e9e8918f9a2..fccf1d046204 (1 commits) (google#690)
206a069 Roll third_party/spirv-headers 9242862c84fe..9cf7c3a7d2d2 (1 commits) (google#689)
2f42850 Roll all DEPS to HEAD (google#685)
facd092 Roll third_party/effcee 8f0a61dc95e0..b83b58d177b7 (2 commits) (google#677)
3c76b8a Re-introduce SetShaderModel for backwards compatibility (google#675)
8738101 Refactor dup'd arg handling functions into a single location (google#660)
f21bd30 Update Dockerfile to use Python 3 (google#668)
d0298a3  Add details of spvc to README.md (google#661)
78c188d Change TODO(name) to TODO(bug issue#) for spvc files (google#658)
dc67137 Fixing duplicate absolute install paths when using Visual Studio generators (google#639)
568cf20 Add in missing flags to pass more of the spirv-cross tests (google#664)
acf7d24 Convert vars['foo'] to Var('foo') (google#663)

Testing: checkbuild.py on Linux; unit tests on Windows

Change-Id: Ic91997e8266c6f53239373657fb1f566a0223bf9
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Factor out duplicated main utility functions function
2 participants