hfinkel accepted this revision.
hfinkel added a comment.
This revision is now accepted and ready to land.

In https://reviews.llvm.org/D35817#835740, @t.p.northover wrote:

> > What's going on with the OpenMP test changes?
>
> Compound assignment operators like "real /= complex" become illegal under the 
> new rules (in C++) because somewhere there has to be an implicit conversion. 
> I was pretty relieved to discover GCC also rejects the syntax because my 
> change actually messes up the AST quite comprehensively if we wanted it to be 
> legal (OpenMP does some weird stuff building its AST).


Okay. LGTM.


https://reviews.llvm.org/D35817



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to