Addressed review comments

The LLVM comments needs to be proper prose. Updated the comments.

Author: rajatbajpai

llvm/unittests/IR/IntrinsicsTest.cpp

@@ -198,7 +198,7 @@ TEST(IntrinsicAttributes, TestGetFnAttributesBug) {
   EXPECT_FALSE(AS.hasAttributes());
 }
 
-// Test non-overloaded intrinsic
+// Tests non-overloaded intrinsic declaration.
 TEST_F(IntrinsicsTest, NonOverloadedIntrinsic) {
   Type *RetTy = Type::getVoidTy(Context);
   SmallVector<Type *, 1> ArgTys;
@@ -215,7 +215,7 @@ TEST_F(IntrinsicsTest, NonOverloadedIntrinsic) {
   EXPECT_EQ(F->getName(), "llvm.assume");
 }
 
-// Test overloaded intrinsic with automatic type resolution (scalar)
+// Tests overloaded intrinsic with automatic type resolution for scalar types.
 TEST_F(IntrinsicsTest, OverloadedIntrinsicScalar) {
   Type *RetTy = Type::getInt32Ty(Context);
   SmallVector<Type *, 2> ArgTys;
@@ -233,7 +233,7 @@ TEST_F(IntrinsicsTest, OverloadedIntrinsicScalar) {
   EXPECT_EQ(F->getName(), "llvm.umax.i32");
 }
 
-// Test overloaded intrinsic with automatic type resolution (vector)
+// Tests overloaded intrinsic with automatic type resolution for vector types.
 TEST_F(IntrinsicsTest, OverloadedIntrinsicVector) {
   Type *RetTy = FixedVectorType::get(Type::getInt32Ty(Context), 4);
   SmallVector<Type *, 2> ArgTys;
@@ -251,7 +251,7 @@ TEST_F(IntrinsicsTest, OverloadedIntrinsicVector) {
   EXPECT_EQ(F->getName(), "llvm.umax.v4i32");
 }
 
-// Test vararg intrinsic: experimental_gc_statepoint
+// Tests vararg intrinsic declaration.
 TEST_F(IntrinsicsTest, VarArgIntrinsicStatepoint) {
   Type *RetTy = Type::getTokenTy(Context);
   SmallVector<Type *, 5> ArgTys;
@@ -272,7 +272,7 @@ TEST_F(IntrinsicsTest, VarArgIntrinsicStatepoint) {
   EXPECT_EQ(F->getName(), "llvm.experimental.gc.statepoint.p0");
 }
 
-// Test different overloads create different declarations
+// Tests that different overloads create different declarations.
 TEST_F(IntrinsicsTest, DifferentOverloads) {
   // i32 version
   Type *RetTy32 = Type::getInt32Ty(Context);
@@ -297,7 +297,7 @@ TEST_F(IntrinsicsTest, DifferentOverloads) {
   EXPECT_EQ(F64->getName(), "llvm.umax.i64");
 }
 
-// Test IRBuilder::CreateIntrinsic with overloaded scalar type
+// Tests IRBuilder::CreateIntrinsic with overloaded scalar type.
 TEST_F(IntrinsicsTest, IRBuilderCreateIntrinsicScalar) {
   IRBuilder<> Builder(BB);
 
@@ -315,7 +315,7 @@ TEST_F(IntrinsicsTest, IRBuilderCreateIntrinsicScalar) {
   EXPECT_FALSE(CI->getCalledFunction()->isVarArg());
 }
 
-// Test IRBuilder::CreateIntrinsic with vector intrinsic
+// Tests IRBuilder::CreateIntrinsic with overloaded vector type.
 TEST_F(IntrinsicsTest, IRBuilderCreateIntrinsicVector) {
   IRBuilder<> Builder(BB);
 
@@ -333,7 +333,7 @@ TEST_F(IntrinsicsTest, IRBuilderCreateIntrinsicVector) {
   EXPECT_FALSE(CI->getCalledFunction()->isVarArg());
 }
 
-// Test IRBuilder::CreateIntrinsic with vararg intrinsic
+// Tests IRBuilder::CreateIntrinsic with vararg intrinsic.
 TEST_F(IntrinsicsTest, IRBuilderCreateIntrinsicVarArg) {
   IRBuilder<> Builder(BB);