Re: [PATCH] D15704: Add some overlooked optnone tests, and tighten up an existing test

2016-01-04 Thread Paul Robinson via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL256762: Add some overlooked optnone tests, and tighten up an existing test. (authored by probinson). Changed prior to commit: http://reviews.llvm.org/D15704?vs=43416&id=43891#toc Repository: rL LLVM

Re: [PATCH] D15704: Add some overlooked optnone tests, and tighten up an existing test

2015-12-27 Thread Aaron Ballman via cfe-commits
aaron.ballman accepted this revision. aaron.ballman added a comment. This revision is now accepted and ready to land. LGTM! Thank you for these; I think the optnon-class-members.cpp and optnon-templates.cpp have value, so I would say to keep them. http://reviews.llvm.org/D15704 _

[PATCH] D15704: Add some overlooked optnone tests, and tighten up an existing test

2015-12-21 Thread Paul Robinson via cfe-commits
probinson created this revision. probinson added a reviewer: aaron.ballman. probinson added a subscriber: cfe-commits. Found some tests we wrote back in the day, but which never made it upstream. optnone-and-attributes.cpp is a more thorough exercise of optnone attribute interaction than we have