Changes

Summary

1. [SelectionDAG] Replace llvm_unreachable at the end of getCopyFromParts with a report_fatal_error. Based on PR41748, not all cases are handled in this function. llvm_unreachable is treated as an optimization hint than can prune code paths in a release build. This causes weird behavior when PR41748 is encountered on a release build. It appears to generate an fp_round instruction from the floating point code. Making this a report_fatal_error prevents incorrect optimization of the code and will instead generate a message to file a bug report.

Revision 360008 by ctopper: [SelectionDAG] Replace llvm_unreachable at the end of getCopyFromParts with a report_fatal_error. Based on PR41748, not all cases are handled in this function. llvm_unreachable is treated as an optimization hint than can prune code paths in a release build. This causes weird behavior when PR41748 is encountered on a release build. It appears to generate an fp_round instruction from the floating point code. Making this a report_fatal_error prevents incorrect optimization of the code and will instead generate a message to file a bug report.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp	trunk/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp

Summary

1. [c++20] Implement P1009R2: allow omitting the array bound in an array new expression. This was voted into C++20 as a defect report resolution, so we retroactively apply it to all prior language modes (though it can never actually be used before C++11 mode).
2. The 'CodeGenObjC/illegal-UTF8.m' get failed with Clang built with 32-bit targets only (as example ARM-only) with the following error: error: unable to create target: 'No available targets are compatible with triple "< ... any 64-bit target triple ... >"' I didn't find any 64-bit dependencies for the test and I think removing '-m64' option should fix the problem and allow this test for any target specified by LLVM_DEFAULT_TARGET_TRIPLE. Patch by Vlad Vereschaka. Differential Revision: https://reviews.llvm.org/D61345

Revision 360006 by rsmith: [c++20] Implement P1009R2: allow omitting the array bound in an array new expression. This was voted into C++20 as a defect report resolution, so we retroactively apply it to all prior language modes (though it can never actually be used before C++11 mode).
Change Type	Path in Repository	Path in Workspace
	/cfe/trunk/include/clang/AST/ExprCXX.h	trunk/include/clang/AST/ExprCXX.h
	/cfe/trunk/include/clang/AST/Stmt.h	trunk/include/clang/AST/Stmt.h
	/cfe/trunk/include/clang/ASTMatchers/ASTMatchers.h	trunk/include/clang/ASTMatchers/ASTMatchers.h
	/cfe/trunk/include/clang/Basic/DiagnosticSemaKinds.td	trunk/include/clang/Basic/DiagnosticSemaKinds.td
	/cfe/trunk/include/clang/Sema/Sema.h	trunk/include/clang/Sema/Sema.h
	/cfe/trunk/lib/AST/ASTImporter.cpp	trunk/lib/AST/ASTImporter.cpp
	/cfe/trunk/lib/AST/ExprCXX.cpp	trunk/lib/AST/ExprCXX.cpp
	/cfe/trunk/lib/AST/StmtPrinter.cpp	trunk/lib/AST/StmtPrinter.cpp
	/cfe/trunk/lib/Analysis/CFG.cpp	trunk/lib/Analysis/CFG.cpp
	/cfe/trunk/lib/CodeGen/CGExprCXX.cpp	trunk/lib/CodeGen/CGExprCXX.cpp
	/cfe/trunk/lib/Parse/ParseExprCXX.cpp	trunk/lib/Parse/ParseExprCXX.cpp
	/cfe/trunk/lib/Sema/SemaExprCXX.cpp	trunk/lib/Sema/SemaExprCXX.cpp
	/cfe/trunk/lib/Sema/TreeTransform.h	trunk/lib/Sema/TreeTransform.h
	/cfe/trunk/lib/StaticAnalyzer/Checkers/MallocChecker.cpp	trunk/lib/StaticAnalyzer/Checkers/MallocChecker.cpp
	/cfe/trunk/test/CodeGenCXX/new-array-init.cpp	trunk/test/CodeGenCXX/new-array-init.cpp
	/cfe/trunk/test/PCH/cxx-exprs.cpp	trunk/test/PCH/cxx-exprs.cpp
	/cfe/trunk/test/SemaCXX/new-delete.cpp	trunk/test/SemaCXX/new-delete.cpp
	/cfe/trunk/tools/libclang/CIndex.cpp	trunk/tools/libclang/CIndex.cpp
	/cfe/trunk/www/cxx_status.html	trunk/www/cxx_status.html
Revision 360005 by gkistanova: The 'CodeGenObjC/illegal-UTF8.m' get failed with Clang built with 32-bit targets only (as example ARM-only) with the following error: error: unable to create target: 'No available targets are compatible with triple "< ... any 64-bit target triple ... >"' I didn't find any 64-bit dependencies for the test and I think removing '-m64' option should fix the problem and allow this test for any target specified by LLVM_DEFAULT_TARGET_TRIPLE. Patch by Vlad Vereschaka. Differential Revision: https://reviews.llvm.org/D61345
Change Type	Path in Repository	Path in Workspace
	/cfe/trunk/test/CodeGenObjC/illegal-UTF8.m	trunk/test/CodeGenObjC/illegal-UTF8.m

Summary

1. Fix up after r360006.

Revision 360007 by rsmith: Fix up after r360006.
Change Type	Path in Repository	Path in Workspace
	/clang-tools-extra/trunk/clang-tidy/modernize/MakeSmartPtrCheck.cpp	trunk/clang-tidy/modernize/MakeSmartPtrCheck.cpp