Project Ortega BCM5719 LLVM https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.1 2019-05-29T16:59:48+00:00 2019-05-29T16:59:48+00:00 Matt Arsenault Matthew.Arsenault@amd.com 2019-05-29T16:59:48+00:00 urn:sha1:f80c4241b306ce4cf2d718b590e516866d787c67 llvm-svn: 361990 2019-05-25T22:34:27+00:00 David Bolvansky david.bolvansky@gmail.com 2019-05-25T22:34:27+00:00 urn:sha1:0290a77aa8609a99ba613efef40ec1626aec362d Summary: PR41688 Reviewers: spatel, efriedma, craig.topper, hfinkel, reames Reviewed By: hfinkel Subscribers: javed.absar, dmgreen, fhahn, hfinkel, reames, nikic, lebedev.ri, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D61409 llvm-svn: 361707 2019-05-25T13:11:22+00:00 David Bolvansky david.bolvansky@gmail.com 2019-05-25T13:11:22+00:00 urn:sha1:bb76cf0f964d00d7673a3d673809618343f308cc llvm-svn: 361695 2019-04-17T04:52:47+00:00 Eric Christopher echristo@gmail.com 2019-04-17T04:52:47+00:00 urn:sha1:cee313d288a4faf0355d76fb6e0e927e211d08a5 The reversion apparently deleted the test/Transforms directory. Will be re-reverting again. llvm-svn: 358552 2019-04-17T02:12:23+00:00 Eric Christopher echristo@gmail.com 2019-04-17T02:12:23+00:00 urn:sha1:a86343512845c9c1fdbac865fea88aa5fce7142a As it's causing some bot failures (and per request from kbarton). This reverts commit r358543/ab70da07286e618016e78247e4a24fcb84077fda. llvm-svn: 358546 2018-11-29T15:27:04+00:00 Joseph Tremoulet jotrem@microsoft.com 2018-11-29T15:27:04+00:00 urn:sha1:926ee459c40dc6e22945439d278f6e4ec553a640 Summary: When splitting musttail calls, the split blocks' original terminators get removed; inform the DTU when this happens. Also add a testcase that fails an assertion in the DTU without this fix. Reviewers: fhahn, junbuml Reviewed By: fhahn Subscribers: llvm-commits Differential Revision: https://reviews.llvm.org/D55027 llvm-svn: 347872 2018-11-14T10:04:30+00:00 Florian Hahn florian.hahn@arm.com 2018-11-14T10:04:30+00:00 urn:sha1:505091a8f2ae676838dc33e429d81e3d2555edf4 The underlying problem causing the expensive-check failure was fixed in rL346769. llvm-svn: 346843 2018-11-13T17:54:43+00:00 Florian Hahn florian.hahn@arm.com 2018-11-13T17:54:43+00:00 urn:sha1:107d0a87565ac7f067e38b1f73866959cad6f52f This patch updates DuplicateInstructionsInSplitBetween to update a DTU instead of applying updates to the DT directly. Given that there only are 2 users, also updated them in this patch to avoid churn. I slightly moved the code in CallSiteSplitting around to reduce the places where we have to pass in DTU. If necessary, I could split those changes in a separate patch. This fixes missing DT updates when dealing with musttail calls in CallSiteSplitting, by using DTU->deleteBB. Reviewers: junbuml, kuhar, NutshellySima, indutny, brzycki Reviewed By: NutshellySima llvm-svn: 346769 2018-11-09T13:28:58+00:00 Florian Hahn florian.hahn@arm.com 2018-11-09T13:28:58+00:00 urn:sha1:9f878e9baef070970b11e8e7220f69ff804650bf This cause a failure with EXPENSIVE_CHECKS llvm-svn: 346492 2018-11-09T10:23:46+00:00 Florian Hahn florian.hahn@arm.com 2018-11-09T10:23:46+00:00 urn:sha1:52578f95c9fcf7b85beea921d401f49b2f6e9d1d We can stop recording conditions once we reached the immediate dominator for the block containing the call site. Conditions in predecessors of the that node will be the same for all paths to the call site and splitting is not beneficial. This patch makes CallSiteSplitting dependent on the DT anlysis. because the immediate dominators seem to be the easiest way of finding the node to stop at. I had to update some exiting tests, because they were checking for conditions that were true/false on all paths to the call site. Those should now be handled by instcombine/ipsccp. Reviewers: davide, junbuml Reviewed By: junbuml Differential Revision: https://reviews.llvm.org/D44627 llvm-svn: 346483