Changes

Summary

[libFuzzer] Fix __sanitizer_print_memory_profile return type in ExtFunctions.def Summary: Looks like a typo, as that function actually returns void and is used as such in libFuzzer code as well. Reviewers: kcc, Dor1s Reviewed By: Dor1s Subscribers: delcypher, #sanitizers, llvm-commits Tags: #llvm, #sanitizers Differential Revision: https://reviews.llvm.org/D65160
[Remarks] Add unit tests for YAML serialization Add tests for both the string table and non string table case.
clang-format: Fix namespace end comments for namespaces with attributes and macros. Fixes PR39247. While here, also make C++20 `namespace A::inline B::inline C` nested inline namespaced definitions work. Before: #define DEPRECATE_WOOF [[deprecated("meow")]] namespace DEPRECATE_WOOF woof { void f() {} } // namespace DEPRECATE_WOOFwoof namespace [[deprecated("meow")]] woof { void f() {} } // namespace [[deprecated("meow")]]woof namespace woof::inline bark { void f() {} } // namespace woof::inlinebark Now: #define DEPRECATE_WOOF [[deprecated("meow")]] namespace DEPRECATE_WOOF woof { void f() {} } // namespace woof namespace [[deprecated("meow")]] woof { void f() {} } // namespace woof namespace woof::inline bark { void f() {} } // namespace woof::inline bark (In addition to the fixed namespace end comments, also note the correct indent of the namespace contents.) Differential Revision: https://reviews.llvm.org/D65125
[IndVars] Fix a subtle bug in optimizeLoopExits The original code failed to account for the fact that one exit can have a pointer exit count without all of them having pointer exit counts. This could cause two separate bugs: 1) We might exit the loop early, and leave optimizations undone. This is what triggered the assertion failure in the reported test case. 2) We might optimize one exit, then exit without indicating a change. This could result in an analysis invalidaton bug if no other transform is done by the rest of indvars. Note that the pointer exit counts are a really fragile concept. They show up only when we have a pointer IV w/o a datalayout to provide their size. It's really questionable to me whether the complexity implied is worth it.
Improve clang-format-diff help output The description in clang-format-diff.py is more useful than the one in `clang-format-diff -h`, so use the same description in both places. Differential Revision: https://reviews.llvm.org/D64998
[IR][Verifier] Allow IntToPtrInst to be !dereferenceable Summary: Allow IntToPtrInst to carry !dereferenceable metadata tag. This is valid since !dereferenceable can be only be applied to pointer type values. Change-Id: If8a6e3c616f073d51eaff52ab74535c29ed497b4 Subscribers: llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D64954
[RISCV][NFC] Correct RUN in rvi-pseudos-invalid.s This test should test both riscv32 and riscv64.

Change Type	Path in Repository	Path in Workspace
Revision 366834 by dor1s: [libFuzzer] Fix __sanitizer_print_memory_profile return type in ExtFunctions.def Summary: Looks like a typo, as that function actually returns void and is used as such in libFuzzer code as well. Reviewers: kcc, Dor1s Reviewed By: Dor1s Subscribers: delcypher, #sanitizers, llvm-commits Tags: #llvm, #sanitizers Differential Revision: https://reviews.llvm.org/D65160
	/compiler-rt/trunk/lib/fuzzer/FuzzerExtFunctions.def (diff)	compiler-rt.src/lib/fuzzer/FuzzerExtFunctions.def
Revision 366832 by thegameg: [Remarks] Add unit tests for YAML serialization Add tests for both the string table and non string table case.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/Remarks/RemarkStringTable.cpp (diff)	llvm.src/lib/Remarks/RemarkStringTable.cpp
	/llvm/trunk/unittests/Remarks/CMakeLists.txt (diff)	llvm.src/unittests/Remarks/CMakeLists.txt
	/llvm/trunk/unittests/Remarks/YAMLRemarksSerializerTest.cpp	llvm.src/unittests/Remarks/YAMLRemarksSerializerTest.cpp
Revision 366831 by nico: clang-format: Fix namespace end comments for namespaces with attributes and macros. Fixes PR39247. While here, also make C++20 `namespace A::inline B::inline C` nested inline namespaced definitions work. Before: #define DEPRECATE_WOOF [[deprecated("meow")]] namespace DEPRECATE_WOOF woof { void f() {} } // namespace DEPRECATE_WOOFwoof namespace [[deprecated("meow")]] woof { void f() {} } // namespace [[deprecated("meow")]]woof namespace woof::inline bark { void f() {} } // namespace woof::inlinebark Now: #define DEPRECATE_WOOF [[deprecated("meow")]] namespace DEPRECATE_WOOF woof { void f() {} } // namespace woof namespace [[deprecated("meow")]] woof { void f() {} } // namespace woof namespace woof::inline bark { void f() {} } // namespace woof::inline bark (In addition to the fixed namespace end comments, also note the correct indent of the namespace contents.) Differential Revision: https://reviews.llvm.org/D65125
Change Type	Path in Repository	Path in Workspace
	/cfe/trunk/lib/Format/NamespaceEndCommentsFixer.cpp (diff)	clang.src/lib/Format/NamespaceEndCommentsFixer.cpp
	/cfe/trunk/lib/Format/UnwrappedLineParser.cpp (diff)	clang.src/lib/Format/UnwrappedLineParser.cpp
	/cfe/trunk/unittests/Format/FormatTest.cpp (diff)	clang.src/unittests/Format/FormatTest.cpp
	/cfe/trunk/unittests/Format/NamespaceEndCommentsFixerTest.cpp (diff)	clang.src/unittests/Format/NamespaceEndCommentsFixerTest.cpp
Revision 366829 by reames: [IndVars] Fix a subtle bug in optimizeLoopExits The original code failed to account for the fact that one exit can have a pointer exit count without all of them having pointer exit counts. This could cause two separate bugs: 1) We might exit the loop early, and leave optimizations undone. This is what triggered the assertion failure in the reported test case. 2) We might optimize one exit, then exit without indicating a change. This could result in an analysis invalidaton bug if no other transform is done by the rest of indvars. Note that the pointer exit counts are a really fragile concept. They show up only when we have a pointer IV w/o a datalayout to provide their size. It's really questionable to me whether the complexity implied is worth it.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/lib/Transforms/Scalar/IndVarSimplify.cpp (diff)	llvm.src/lib/Transforms/Scalar/IndVarSimplify.cpp
	/llvm/trunk/test/Transforms/IndVarSimplify/eliminate-exit-no-dl.ll	llvm.src/test/Transforms/IndVarSimplify/eliminate-exit-no-dl.ll
Revision 366828 by nico: Improve clang-format-diff help output The description in clang-format-diff.py is more useful than the one in `clang-format-diff -h`, so use the same description in both places. Differential Revision: https://reviews.llvm.org/D64998
Change Type	Path in Repository	Path in Workspace
	/cfe/trunk/tools/clang-format/clang-format-diff.py (diff)	clang.src/tools/clang-format/clang-format-diff.py
Revision 366826 by rtayl: [IR][Verifier] Allow IntToPtrInst to be !dereferenceable Summary: Allow IntToPtrInst to carry !dereferenceable metadata tag. This is valid since !dereferenceable can be only be applied to pointer type values. Change-Id: If8a6e3c616f073d51eaff52ab74535c29ed497b4 Subscribers: llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D64954
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/docs/LangRef.rst (diff)	llvm.src/docs/LangRef.rst
	/llvm/trunk/lib/IR/Value.cpp (diff)	llvm.src/lib/IR/Value.cpp
	/llvm/trunk/lib/IR/Verifier.cpp (diff)	llvm.src/lib/IR/Verifier.cpp
	/llvm/trunk/test/Analysis/ValueTracking/memory-dereferenceable.ll (diff)	llvm.src/test/Analysis/ValueTracking/memory-dereferenceable.ll
	/llvm/trunk/test/Verifier/dereferenceable-md-inttoptr.ll	llvm.src/test/Verifier/dereferenceable-md-inttoptr.ll
	/llvm/trunk/test/Verifier/dereferenceable-md.ll (diff)	llvm.src/test/Verifier/dereferenceable-md.ll
Revision 366825 by lenary: [RISCV][NFC] Correct RUN in rvi-pseudos-invalid.s This test should test both riscv32 and riscv64.
Change Type	Path in Repository	Path in Workspace
	/llvm/trunk/test/MC/RISCV/rvi-pseudos-invalid.s (diff)	llvm.src/test/MC/RISCV/rvi-pseudos-invalid.s