commit 7ee30f5d2f5ac77f9d856560a4971055854285ec
author James Price <jrprice@google.com> Wed Aug 21 01:40:04 2024 +0000
committer Dawn LUCI CQ <dawn-scoped@luci-project-accounts.iam.gserviceaccount.com> Wed Aug 21 01:40:04 2024 +0000
tree 9369945e1e117c6e6634307662e2d5cfe7a4528e
parent 7ce7b5be549cf51d25ff0f38a6e298cb45b4deb6

[tint] Update insertion point in InsertAfter()

When inserting multiple instructions using `InsertAfter()`, it's
reasonable to expect that the instructions created inside the lambda
retain the order that they are created in. This change makes that the
case by updating the insertion point after inserting each instruction.

Add some basic tests for the insertion helpers.

Change-Id: I0eb850675dd9a136818a6b4cadbb63126363ab54
Reviewed-on: https://dawn-review.googlesource.com/c/dawn/+/203316
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Commit-Queue: James Price <jrprice@google.com>

src/tint/lang/core/ir/BUILD.bazel

diff --git a/src/tint/lang/core/ir/BUILD.bazel b/src/tint/lang/core/ir/BUILD.bazel
index 99c054a..81ee19e 100644
--- a/src/tint/lang/core/ir/BUILD.bazel
+++ b/src/tint/lang/core/ir/BUILD.bazel
@@ -180,6 +180,7 @@
     "block_param_test.cc",
     "block_test.cc",
     "break_if_test.cc",
+    "builder_test.cc",
     "constant_test.cc",
     "construct_test.cc",
     "continue_test.cc",