en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp#878a24ee Reapply "Fix crash on switch conditions of non-integer types in templates"This patch reapplies commit 759948467ea. Patch was reverted due to aclang-tidy test fail on Windows. The test has been modified. Thereare no additional code changes.Patch was tested with ninja check-all on Windows and Linux.Summary of code changes:Clang currently crashes for switch statements inside a template when thecondition is a non-integer field member because contextual implicitconversion is skipped when parsing the condition. This conversion ishowever later checked in an assert when the case statement is handled.The conversion is skipped when parsing the condition becausethe field member is set as type-dependent based on its containing class.This patch sets the type dependency based on the field's type instead.This patch fixes Bug 40982. List of files: /llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp Tue, 03 Dec 2019 22:20:52 +0000 Elizabeth Andrews <elizabeth.andrews@intel.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp#75994846 Reapply "Fix crash on switch conditions of non-integer types in templates"This patch reapplies commit 76945821b9cad3. The first version brokebuildbots due to clang-tidy test fails. The fails are because someerrors in templates are now diagnosed earlier (does not wait tillinstantiation). I have modified the tests to add checks for thesediagnostics/prevent these diagnostics. There are no additional codechanges.Summary of code changes:Clang currently crashes for switch statements inside a template when thecondition is a non-integer field member because contextual implicitconversion is skipped when parsing the condition. This conversion ishowever later checked in an assert when the case statement is handled.The conversion is skipped when parsing the condition becausethe field member is set as type-dependent based on its containing class.This patch sets the type dependency based on the field's type instead.This patch fixes Bug 40982.Reviewers: rnk, gribozavr2Patch by: Elizabeth Andrews (eandrews)Differential revision: https://reviews.llvm.org/D69950 List of files: /llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp Fri, 08 Nov 2019 18:14:51 +0000 Melanie Blower <melanie.blower@intel.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp#76945821 Fix crash on switch conditions of non-integer types in templatesClang currently crashes for switch statements inside a template whenthe condition is a non-integer field. The crash is due to incorrecttype-dependency of field. Type-dependency of member expressions iscurrently set based on the containing class. This patch changes this for'members of the current instantiation' to set the type dependency basedon the member's type instead.A few lit tests started to fail once I applied this patch because errorsare now diagnosed earlier (does not wait till instantiation). I've modifiedthese tests in this patch as well.Patch fixes PR#40982Differential Revision: https://reviews.llvm.org/D61027llvm-svn: 368706 List of files: /llvm-project-15.0.7/clang/test/SemaTemplate/non-integral-switch-cond.cpp Tue, 13 Aug 2019 15:53:19 +0000 Elizabeth Andrews <elizabeth.andrews@intel.com>