rjmccall added inline comments.

================
Comment at: lib/Sema/SemaCast.cpp:535
+    T1 = Unwrap(T1);
+    T2 = Unwrap(T2).withCVRQualifiers(T2.getCVRQualifiers());
+  }
----------------
Hmm.  Just CVR?  I understand that we can have problems here with the 
enumerated qualifiers, so maybe we shouldn't blindly merge them, but is there 
some way to propagate them if not conflicting?


Repository:
  rC Clang

https://reviews.llvm.org/D49457



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D49457: D... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D494... John McCall via Phabricator via cfe-commits
    • [PATCH] D494... Richard Smith - zygoloid via Phabricator via cfe-commits
    • [PATCH] D494... John McCall via Phabricator via cfe-commits
    • [PATCH] D494... David Zarzycki via Phabricator via cfe-commits
    • [PATCH] D494... Richard Smith - zygoloid via Phabricator via cfe-commits

Reply via email to