sdardis closed this revision.
sdardis added a comment.
Committed as https://reviews.llvm.org/rL281610.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sdardis added a comment.
Actually, seeing your reply on the related patch, keep the comment as it
reflects trunk.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo
sdardis added a comment.
Yes, I think you should also reflect mpf's comment here too.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sdardis accepted this revision.
sdardis added a comment.
This revision is now accepted and ready to land.
LGTM.
https://reviews.llvm.org/D22679
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cf
