Remove unnecessary isImageReadoOnly(), isImageWriteOnly(), & isImageReadWrite calls

[brianagrace]

Summary

This is repetition of isImage() function in NVPTXUtilities.cpp.

[jlebar]

Thank you for the patch!

I'll check this in for you when I have a chance, hopefully later today.

lib/Target/NVPTX/NVPTXImageOptimizer.cpp

 }
 
 bool NVPTXImageOptimizer::replaceIsTypePSampler(Instruction &I) {
   Value *TexHandle = cleanupValue(I.getOperand(0));
   if (isSampler(*TexHandle)) {
     // This is an OpenCL sampler, so it must be a samplerref
     replaceWith(&I, ConstantInt::getTrue(I.getContext()));
     return true;
-  } else if (isImageWriteOnly(*TexHandle) ||
-             isImageReadWrite(*TexHandle) ||
-             isImageReadOnly(*TexHandle)) {
+  } else if (isImage(*TexHandle)) {
     // This is an OpenCL image, so it cannot be a samplerref
     replaceWith(&I, ConstantInt::getFalse(I.getContext()));
     return true;
   } else {
     // The image type is unknown, so we cannot eliminate the intrinsic
     return false;
   }
 }