document the function declaration and change param name

Author: changkhothuychung

clang/include/clang/Parse/Parser.h

@@ -5158,6 +5158,16 @@ class Parser : public CodeCompletionHandler {
 
   ///@}
 
+  //===--------------------------------------------------------------------===//
+  // Reflection parsing
+
+  /// ParseCXXReflectExpression - parses the operand of reflection operator
+  /// \param DoubleCaretLoc the location of the double caret operator
+  ///
+  /// \returns on success, a expression holdiing the constructed CXXReflectExpr;
+  ///          on failture, an ExprError
+  ExprResult ParseCXXReflectExpression(SourceLocation DoubleCaretLoc);
+
   //
   //
   // -------------------------------------------------------------------------
@@ -5169,8 +5179,6 @@ class Parser : public CodeCompletionHandler {
   ///@{
 
   //===--------------------------------------------------------------------===//
-  // Parses the operand of reflection operator
-  ExprResult ParseCXXReflectExpression(SourceLocation OpLoc);
 
 private:
   bool MaybeParseHLSLAnnotations(Declarator &D,
@@ -7682,7 +7690,7 @@ class Parser : public CodeCompletionHandler {
   /// [GNU] asm-clobbers:
   ///         asm-string-literal
   ///         asm-clobbers ',' asm-string-literal
-  /// \endverbatim 
+  /// \endverbatim
   ///
   StmtResult ParseAsmStatement(bool &msAsm);
 

clang/lib/Parse/ParseReflect.cpp

@@ -16,7 +16,7 @@
 #include "clang/Sema/EnterExpressionEvaluationContext.h"
 using namespace clang;
 
-ExprResult Parser::ParseCXXReflectExpression(SourceLocation OpLoc) {
+ExprResult Parser::ParseCXXReflectExpression(SourceLocation DoubleCaretLoc) {
   // TODO(reflection) : support parsing for more reflect-expressions.
   EnterExpressionEvaluationContext Unevaluated(
       Actions, Sema::ExpressionEvaluationContext::Unevaluated);
@@ -30,7 +30,8 @@ ExprResult Parser::ParseCXXReflectExpression(SourceLocation OpLoc) {
       ConsumeToken();
       TPA.Commit();
       Decl *TUDecl = Actions.getASTContext().getTranslationUnitDecl();
-      return Actions.ActOnCXXReflectExpr(OpLoc, SourceLocation(), TUDecl);
+      return Actions.ActOnCXXReflectExpr(DoubleCaretLoc, SourceLocation(),
+                                         TUDecl);
     }
     TPA.Revert();
   }
@@ -52,7 +53,7 @@ ExprResult Parser::ParseCXXReflectExpression(SourceLocation OpLoc) {
     QualType Canon = QT.getCanonicalType();
     if (Canon->isBuiltinType()) {
       // Only supports builtin types for now
-      return Actions.ActOnCXXReflectExpr(OpLoc, TSI);
+      return Actions.ActOnCXXReflectExpr(DoubleCaretLoc, TSI);
     }
   }