From: peter klausler <pklausler@nvidia.com>
Date: Fri, 26 Oct 2018 22:55:28 +0000 (-0700)
Subject: [flang] clean up comments, put out for review
X-Git-Tag: llvmorg-12-init~9537^2~2021
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=8832f8b1441f2754ddfc07af37495733909f3116;p=platform%2Fupstream%2Fllvm.git

[flang] clean up comments, put out for review

Original-commit: flang-compiler/f18@26a3221fc40ca0c3047c11b731426d270e072880
Reviewed-on: https://github.com/flang-compiler/f18/pull/219
Tree-same-pre-rewrite: false
---

diff --git a/flang/lib/evaluate/call.h b/flang/lib/evaluate/call.h
index d7737b0..2ac4cd7 100644
--- a/flang/lib/evaluate/call.h
+++ b/flang/lib/evaluate/call.h
@@ -45,7 +45,6 @@ struct ActualArgument {
   bool isAlternateReturn{false};  // when true, "value" is a label number
 
   // TODO: Mark legacy %VAL and %REF arguments
-  // TODO: pmk: Mark arguments that were originally parenthesized
 
   // Subtlety: There is a distinction that must be maintained here between an
   // actual argument expression that is a variable and one that is not,
diff --git a/flang/lib/evaluate/tools.h b/flang/lib/evaluate/tools.h
index 1cf7e9e..7b8608d 100644
--- a/flang/lib/evaluate/tools.h
+++ b/flang/lib/evaluate/tools.h
@@ -110,7 +110,7 @@ std::optional<std::int64_t> ToInt64(const std::optional<A> &x) {
   }
 }
 
-// TODO pmk: GetSymbol and Rank and GetType here, too
+// TODO: GetSymbol and Rank and GetType here, too
 
 // Generalizing packagers: these take operations and expressions of more
 // specific types and wrap them in Expr<> containers of more abstract types.