[PATCH] D33833: Fix PR 33189: Clang assertion on template destructor declaration

2017-06-26 Thread Kuang He via Phabricator via cfe-commits
kuang_he updated this revision to Diff 104018. https://reviews.llvm.org/D33833 Files: lib/AST/DeclCXX.cpp test/SemaTemplate/destructor-template.cpp Index: test/SemaTemplate/destructor-template.cpp === --- test/SemaTemplate/dest

[PATCH] D33833: Fix PR 33189: Clang assertion on template destructor declaration

2017-06-23 Thread Kuang He via Phabricator via cfe-commits
kuang_he added a comment. Can we get this patch reviewed by any chance? https://reviews.llvm.org/D33833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D33833: Fix PR 33189: Clang assertion on template destructor declaration

2017-06-13 Thread Kuang He via Phabricator via cfe-commits
kuang_he added a comment. Can we have this patch reviewed? https://reviews.llvm.org/D33833 ___ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[PATCH] D33833: Fix PR 33189: Clang assertion on template destructor declaration

2017-06-06 Thread Kuang He via Phabricator via cfe-commits
kuang_he updated this revision to Diff 101601. kuang_he added a comment. Patch updated addressing comment. https://reviews.llvm.org/D33833 Files: lib/AST/DeclCXX.cpp test/SemaCXX/PR33189.cpp Index: test/SemaCXX/PR33189.cpp ==

[PATCH] D33833: Fix Clang assertion on template destructor declaration

2017-06-02 Thread Kuang He via Phabricator via cfe-commits
kuang_he created this revision. This patch aim to fix bug reported at https://bugs.llvm.org/show_bug.cgi?id=33189. Clang hit assertion on template destructor declaration https://reviews.llvm.org/D33833 Files: lib/AST/DeclCXX.cpp test/SemaCXX/PR33189.cpp Index: test/SemaCXX/PR33189.cpp =

[PATCH] D30415: Fix -mno-altivec cannot overwrite -maltivec option

2017-03-27 Thread Kuang He via Phabricator via cfe-commits
kuang_he abandoned this revision. kuang_he added a comment. In https://reviews.llvm.org/D30415#703652, @uweigand wrote: > In https://reviews.llvm.org/D30415#703442, @hfinkel wrote: > > > In https://reviews.llvm.org/D30415#703398, @echristo wrote: > > > > > Different suggestion: > > > > > > Remove

[PATCH] D30415: Fix -mno-altivec cannot overwrite -maltivec option

2017-03-07 Thread Kuang He via Phabricator via cfe-commits
kuang_he updated this revision to Diff 90927. kuang_he edited the summary of this revision. kuang_he added a comment. Add fix and update test case for -fzvector option. https://reviews.llvm.org/D30415 Files: lib/Driver/Tools.cpp test/Driver/ppc-features.cpp test/Driver/systemz-features.cp