tree 7734c13a6248c4c6bfbcfa2b5d98105e544a058e
parent 2bd95f1cf45451119b6644e41919b7bef443d912
author Ben Clayton <bclayton@google.com> 1610441431 +0000
committer Commit Bot service account <commit-bot@chromium.org> 1610441431 +0000

ShaderModule: Refactor ParseWGSL() to take a File*

The tint::ast::Module holds a diagnostic list (tint::diag::List) which references the source tint::Source::File.
If you try to enable any of the more pretty diagnostic printing functionality, and attempt to print these after ParseWGSL() has returned, you'll then dereference a pointer to the now stack unwound `tint::Source::File`.
Promote the file up one callstack level to fix this.

Bug: none. Only exposed when using pretty printing.
Change-Id: I9432dd9d668fd1d92efa228bb5ed31278fd3ddfc
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/37280
Reviewed-by: Corentin Wallez <cwallez@chromium.org>
Reviewed-by: Austin Eng <enga@chromium.org>
Commit-Queue: Corentin Wallez <cwallez@chromium.org>
Auto-Submit: Ben Clayton <bclayton@google.com>