[PATCH] D64477: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest.

2019-07-15 Thread Balázs Kéri via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL366061: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest. (authored by balazske, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CH

[PATCH] D64477: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest.

2019-07-15 Thread Gabor Marton via Phabricator via cfe-commits
martong added a comment. In D64477#1585218 , @balazske wrote: > In D64477#1585092 , @martong wrote: > > > Does it really matter if it is CXX11 or CXX14, in the child patch we use a > > CXX11 using directive. Anyway

[PATCH] D64477: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest.

2019-07-15 Thread Balázs Kéri via Phabricator via cfe-commits
balazske added a comment. In D64477#1585092 , @martong wrote: > Does it really matter if it is CXX11 or CXX14, in the child patch we use a > CXX11 using directive. Anyway, CXX14 is more future proof. Later there should follow the test with variable temp

[PATCH] D64477: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest.

2019-07-15 Thread Gabor Marton via Phabricator via cfe-commits
martong accepted this revision. martong added a comment. Does it really matter if it is CXX11 or CXX14, in the child patch we use a CXX11 using directive. Anyway, CXX14 is more future proof. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64477/new/ https://review

[PATCH] D64477: [ASTImporter] Using Lang_CXX14 in ASTImporterVisibilityTest.

2019-07-15 Thread Balázs Kéri via Phabricator via cfe-commits
balazske updated this revision to Diff 209768. balazske added a comment. - Change to Lang_CXX14. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D64477/new/ https://reviews.llvm.org/D64477 Files: unittests/AST/ASTImporterVisibilityTest.cpp Index: unittests/AST/