SuccessChanges

Summary

  1. Commit missing part of "Split many_tls_keys.cpp into two tests" https://reviews.llvm.org/D67428 This change was lost due to a file rename and modification.
  2. [InstCombine] remove unneeded one-use checks for icmp fold This fold and several others were added in: rL125734 ...with no explanation for the one-use checks other than the code comments about register pressure. Given that this is IR canonicalization, we shouldn't be worried about register pressure though; the backend should be able to adjust for that as needed. There are similar checks as noted with the TODO comments. I'm hoping to remove those restrictions too, but if any of these does cause a regression, it should be easier to correct by making small, individual commits. This is part of solving PR43310 the theoretically right way: https://bugs.llvm.org/show_bug.cgi?id=43310 ...ie, if we don't cripple basic transforms, then we won't need to add special-case code to detect larger patterns.
Revision 371941 by kamil:
Commit missing part of "Split many_tls_keys.cpp into two tests"

https://reviews.llvm.org/D67428

This change was lost due to a file rename and modification.
Change TypePath in RepositoryPath in Workspace
The file was modified/compiler-rt/trunk/test/lsan/TestCases/many_tls_keys_pthread.cpp (diff)compiler-rt.src/test/lsan/TestCases/many_tls_keys_pthread.cpp
The file was modified/compiler-rt/trunk/test/lsan/TestCases/many_tls_keys_thread.cpp (diff)compiler-rt.src/test/lsan/TestCases/many_tls_keys_thread.cpp
Revision 371940 by spatel:
[InstCombine] remove unneeded one-use checks for icmp fold

This fold and several others were added in:
rL125734
...with no explanation for the one-use checks other than the code
comments about register pressure.

Given that this is IR canonicalization, we shouldn't be worried
about register pressure though; the backend should be able to
adjust for that as needed.

There are similar checks as noted with the TODO comments. I'm
hoping to remove those restrictions too, but if any of these
does cause a regression, it should be easier to correct by making
small, individual commits.

This is part of solving PR43310 the theoretically right way:
https://bugs.llvm.org/show_bug.cgi?id=43310
...ie, if we don't cripple basic transforms, then we won't
need to add special-case code to detect larger patterns.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Transforms/InstCombine/InstCombineCompares.cpp (diff)llvm.src/lib/Transforms/InstCombine/InstCombineCompares.cpp
The file was modified/llvm/trunk/test/Transforms/InstCombine/icmp.ll (diff)llvm.src/test/Transforms/InstCombine/icmp.ll