From 5c34060a4e9216b10edd24a01b3ad6329a9bd25f Mon Sep 17 00:00:00 2001
From: Ben Clayton <bclayton@google.com>
Date: Tue, 13 Apr 2021 23:46:17 +0000
Subject: [PATCH] transform/hlsl: Use Symbols().New() for empty entry point

And drop the leading understore, it's no longer needed.

Bug tint:712

Change-Id: Ic0ad304119ceb148984d2fa0a5e9e61f2c3a89fd
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/47637
Commit-Queue: Ben Clayton <bclayton@google.com>
Reviewed-by: Antonio Maiorano <amaiorano@google.com>
---
 src/transform/hlsl.cc      | 3 ++-
 src/transform/hlsl_test.cc | 2 +-
 2 files changed, 3 insertions(+), 2 deletions(-)

diff --git a/src/transform/hlsl.cc b/src/transform/hlsl.cc
index 0da3013e4d..5972a829e5 100644
--- a/src/transform/hlsl.cc
+++ b/src/transform/hlsl.cc
@@ -698,7 +698,8 @@ void Hlsl::AddEmptyEntryPoint(CloneContext& ctx) const {
     }
   }
   ctx.dst->Func(
-      "_tint_unused_entry_point", {}, ctx.dst->ty.void_(), {},
+      ctx.dst->Symbols().New("tint_unused_entry_point"), {},
+      ctx.dst->ty.void_(), {},
       {ctx.dst->create<ast::StageDecoration>(ast::PipelineStage::kVertex)});
 }
 
diff --git a/src/transform/hlsl_test.cc b/src/transform/hlsl_test.cc
index 471ece172b..1f1515bb29 100644
--- a/src/transform/hlsl_test.cc
+++ b/src/transform/hlsl_test.cc
@@ -284,7 +284,7 @@ TEST_F(HlslTest, AddEmptyEntryPoint) {
 
   auto* expect = R"(
 [[stage(vertex)]]
-fn _tint_unused_entry_point() {
+fn tint_unused_entry_point() {
 }
 )";