DEPS: Update Dawn standalone toolchains

Roll chromium/src/build       c7876b5 .. 87b04ad
Roll chromium/src/buildtools  e1471b2 .. f0d740e
Roll chromium/src/tools/clang df9b14e .. 3c4a622

Required to roll ANGLE as it now uses new Clang flags.

Change-Id: I55caa9892ff65ebc4852b2d49df88908ee1ca3e7
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/88221
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Kokoro: Kokoro <noreply+kokoro@google.com>
Commit-Queue: Ben Clayton <bclayton@google.com>
This commit is contained in:
Ben Clayton 2022-04-28 08:26:23 +00:00 committed by Dawn LUCI CQ
parent 928a7d1e93
commit 6bdd6592d9
3 changed files with 3 additions and 7 deletions

6
DEPS
View File

@ -31,11 +31,11 @@ vars = {
deps = {
# Dependencies required to use GN/Clang in standalone
'build': {
'url': '{chromium_git}/chromium/src/build@c7876b5a44308b94074287939244bc562007de69',
'url': '{chromium_git}/chromium/src/build@87b04ad66530e4a571cef36d6e71ef737d23a887',
'condition': 'dawn_standalone',
},
'buildtools': {
'url': '{chromium_git}/chromium/src/buildtools@e1471b21ee9c6765ee95e9db0c76fe997ccad35c',
'url': '{chromium_git}/chromium/src/buildtools@f0d740e4e2f803e39dfd5d8d11f7d87bdf489514',
'condition': 'dawn_standalone',
},
'buildtools/clang_format/script': {
@ -78,7 +78,7 @@ deps = {
},
'tools/clang': {
'url': '{chromium_git}/chromium/src/tools/clang@df9b14e26c163dd8e2c0ab081e2689f038ae7141',
'url': '{chromium_git}/chromium/src/tools/clang@3c4a622d9f0b0ce5ec2a438189d46c695216b324',
'condition': 'dawn_standalone',
},
'tools/clang/dsymutil': {

View File

@ -950,10 +950,8 @@ bool Validator::Function(const sem::Function* func,
return false;
}
auto workgroup_attr_count = 0;
for (auto* attr : decl->attributes) {
if (attr->Is<ast::WorkgroupAttribute>()) {
workgroup_attr_count++;
if (decl->PipelineStage() != ast::PipelineStage::kCompute) {
AddError(
"the workgroup_size attribute is only valid for compute stages",

View File

@ -2262,14 +2262,12 @@ uint32_t Builder::GenerateFunctionCall(const sem::Call* call,
}
ops.push_back(Operand::Int(func_id));
size_t arg_idx = 0;
for (auto* arg : expr->args) {
auto id = GenerateExpressionWithLoadIfNeeded(arg);
if (id == 0) {
return 0;
}
ops.push_back(Operand::Int(id));
arg_idx++;
}
if (!push_function_inst(spv::Op::OpFunctionCall, std::move(ops))) {