[PATCH] D31183: [OpenCL] Added parsing for OpenCL vector types.

Anastasia Stulova via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 24 12:35:11 PDT 2017


Anastasia added a comment.

In https://reviews.llvm.org/D31183#709566, @echuraev wrote:

> In https://reviews.llvm.org/D31183#708833, @yaxunl wrote:
>
> > I think this is a good feature for the convenience of user. I've seen usage like this.
>
>
> I agree. I don't see any reasons why this case doesn't have the right to exist. I don't think that using extra parenthesis is a good solution for solving this problem.


I am just saying that I don't see a big use case for this. I am guessing it can largely come from the macro expansions, but those are generally good style to parenthesize.


https://reviews.llvm.org/D31183





More information about the cfe-commits mailing list