From f16dfeb01544d472a88991c59539d8b704c87e2b Mon Sep 17 00:00:00 2001
From: David Neto <dneto@google.com>
Date: Wed, 11 Nov 2020 14:30:25 +0000
Subject: [PATCH] ast: intrinsic spelling matches the spec

Update the comments around the stream output method for ast::Intrinsic.

Change-Id: I06a3082927c83eb9172357c34a0587aac5f80465
Reviewed-on: https://dawn-review.googlesource.com/c/tint/+/32360
Reviewed-by: dan sinclair <dsinclair@chromium.org>
Commit-Queue: dan sinclair <dsinclair@chromium.org>
---
 src/ast/intrinsic.cc | 2 ++
 src/ast/intrinsic.h  | 2 ++
 2 files changed, 4 insertions(+)

diff --git a/src/ast/intrinsic.cc b/src/ast/intrinsic.cc
index ce3d692215..14fdeefe54 100644
--- a/src/ast/intrinsic.cc
+++ b/src/ast/intrinsic.cc
@@ -18,6 +18,8 @@ namespace tint {
 namespace ast {
 
 std::ostream& operator<<(std::ostream& out, Intrinsic i) {
+  /// The emitted name matches the spelling in the WGSL spec.
+  /// including case.
   switch (i) {
     case Intrinsic::kAbs:
       out << "abs";
diff --git a/src/ast/intrinsic.h b/src/ast/intrinsic.h
index 60fc4acae3..9eb160048f 100644
--- a/src/ast/intrinsic.h
+++ b/src/ast/intrinsic.h
@@ -93,6 +93,8 @@ enum class Intrinsic {
   kTrunc
 };
 
+/// Emits the name of the intrinsic function. The spelling,
+/// including case, matches the name in the WGSL spec.
 std::ostream& operator<<(std::ostream& out, Intrinsic i);
 
 namespace intrinsic {