Re: [PATCH] D23821: Minor cleanup of the class Pattern

2016-08-26 Thread Saleem Abdulrasool via cfe-commits
compnerd added a subscriber: compnerd. compnerd closed this revision. compnerd added a comment. SVN r279832 Repository: rL LLVM https://reviews.llvm.org/D23821 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/

Re: [PATCH] D23821: Minor cleanup of the class Pattern

2016-08-23 Thread Vedant Kumar via cfe-commits
vsk added a subscriber: vsk. vsk accepted this revision. vsk added a reviewer: vsk. vsk added a comment. This revision is now accepted and ready to land. I couldn't find any sites which rely on implicit construction of Pattern after a quick scan. The field re-ordering change also looks innocuous.

[PATCH] D23821: Minor cleanup of the class Pattern

2016-08-23 Thread Alexander Shaposhnikov via cfe-commits
alexshap created this revision. alexshap added reviewers: dsanders, probinson. alexshap added a subscriber: cfe-commits. alexshap set the repository for this revision to rL LLVM. alexshap changed the visibility of this Differential Revision from "Public (No Login Required)" to "All Users". Herald