Project Ortega BCM5719 LLVM https://git.raptorcs.com/git/bcm5719-llvm/atom?h=meklort-10.0.1 2019-10-17T00:37:04+00:00 2019-10-17T00:37:04+00:00 Daniel Sanders daniel_l_sanders@apple.com 2019-10-17T00:37:04+00:00 urn:sha1:329e748c8c3b0fd3942cb1dafa9e4d0bb0aea8a4 Summary: Each generated helper can be configured to generate an option that disables rules in that helper. This can be used to bisect rulesets. The disable bits are stored in a SparseVector as this is very cheap for the common case where nothing is disabled. It gets more expensive the more rules are disabled but you're generally doing that for debug purposes where performance is less of a concern. Depends on D68426 Reviewers: volkan, bogner Reviewed By: volkan Subscribers: hiraditya, Petar.Avramovic, llvm-commits Tags: #llvm Differential Revision: https://reviews.llvm.org/D68438 llvm-svn: 375067 2019-08-15T15:54:37+00:00 Jonas Devlieghere jonas@devlieghere.com 2019-08-15T15:54:37+00:00 urn:sha1:0eaee545eef49ff9498234d3a51a5cbde59bf976 Now that we've moved to C++14, we no longer need the llvm::make_unique implementation from STLExtras.h. This patch is a mechanical replacement of (hopefully) all the llvm::make_unique instances across the monorepo. llvm-svn: 369013 2019-02-15T01:37:54+00:00 Aditya Nandakumar aditya_nandakumar@apple.com 2019-02-15T01:37:54+00:00 urn:sha1:0e362ec19a7e20f4ea53f960ee578e0d5fa1b2e5 https://reviews.llvm.org/D58073 Speed up insertion during the initial populating phase into the GISelWorkList by deferring repeatedly resizing the DenseMap. This results in ~10% improvement in the combiner passes, and ~3% speedup in the Legalizer. reviewed by: aemerson. llvm-svn: 354093 2019-02-11T20:45:19+00:00 Daniel Sanders daniel_l_sanders@apple.com 2019-02-11T20:45:19+00:00 urn:sha1:24e0af69066e1d5ab0a6b4d75ff81d57f5b02f18 The API indicates that the MI is about to be erased rather than it has been erased. llvm-svn: 353746 2019-01-19T08:50:56+00:00 Chandler Carruth chandlerc@gmail.com 2019-01-19T08:50:56+00:00 urn:sha1:2946cd701067404b99c39fb29dc9c74bd7193eb3 to reflect the new license. We understand that people may be surprised that we're moving the header entirely to discuss the new license. We checked this carefully with the Foundation's lawyer and we believe this is the correct approach. Essentially, all code in the project is now made available by the LLVM project under our new license, so you will see that the license headers include that license only. Some of our contributors have contributed code under our old license, and accordingly, we have retained a copy of our old license notice in the top-level files in each project and repository. llvm-svn: 351636 2019-01-16T00:40:37+00:00 Aditya Nandakumar aditya_nandakumar@apple.com 2019-01-16T00:40:37+00:00 urn:sha1:500e3ead9fa5cf27e8dcadae509ffbe93ee0d3bb https://reviews.llvm.org/D52803 This patch adds support to continuously CSE instructions during each of the GISel passes. It consists of a GISelCSEInfo analysis pass that can be used by the CSEMIRBuilder. llvm-svn: 351283 2018-12-14T17:50:14+00:00 Daniel Sanders daniel_l_sanders@apple.com 2018-12-14T17:50:14+00:00 urn:sha1:629db5d8e5a63a2fe533caab0bf019b62a2ab5be Summary: This allows us to register it with the MachineFunction delegate and be notified automatically about erasure and creation of instructions. However, we still need explicit notification for modifications such as those caused by setReg() or replaceRegWith(). There is a catch with this though. The notification for creation is delivered before any operands can be added. While appropriate for scheduling combiner work. This is unfortunate for debug output since an opcode by itself doesn't provide sufficient information on what happened. As a result, the work list remembers the instructions (when debug output is requested) and emits a more complete dump later. Another nit is that the MachineFunction::Delegate provides const pointers which is inconvenient since we want to use it to schedule future modification. To resolve this GISelWorkList now has an optional pointer to the MachineFunction which describes the scope of the work it is permitted to schedule. If a given MachineInstr* is in this function then it is permitted to schedule work to be performed on the MachineInstr's. An alternative to this would be to remove the const from the MachineFunction::Delegate interface, however delegates are not permitted to modify the MachineInstr's they receive. In addition to this, the observer has three interface changes. * erasedInstr() is now erasingInstr() to indicate it is about to be erased but still exists at the moment. * changingInstr() and changedInstr() have been added to report changes before and after they are made. This allows us to trace the changes in the debug output. * As a convenience changingAllUsesOfReg() and finishedChangingAllUsesOfReg() will report changingInstr() and changedInstr() for each use of a given register. This is primarily useful for changes caused by MachineRegisterInfo::replaceRegWith() With this in place, both combine rules have been updated to report their changes to the observer. Finally, make some cosmetic changes to the debug output and make Combiner and CombinerHelp Reviewers: aditya_nandakumar, bogner, volkan, rtereshin, javed.absar Reviewed By: aditya_nandakumar Subscribers: mgorny, rovka, kristof.beyls, llvm-commits Differential Revision: https://reviews.llvm.org/D52947 llvm-svn: 349167 2018-12-12T23:48:13+00:00 Daniel Sanders daniel_l_sanders@apple.com 2018-12-12T23:48:13+00:00 urn:sha1:d001e0e0f47c4f53f4212b49f24cffb1bb4904dd Summary: In addition to knowing that an instruction is changed. It's also useful to know when it's about to change. For example, it might print the instruction so you can track the changes in a debug log, it might remove it from some queue while it's being worked on, or it might want to change several instructions as a single transaction and act on all the changes at once. Added changingInstr() to all existing uses of changedInstr() Reviewers: aditya_nandakumar Reviewed By: aditya_nandakumar Subscribers: rovka, kristof.beyls, llvm-commits Differential Revision: https://reviews.llvm.org/D55623 llvm-svn: 348992 2018-12-12T21:32:01+00:00 Daniel Sanders daniel_l_sanders@apple.com 2018-12-12T21:32:01+00:00 urn:sha1:91dfdd5734e1c0a603e17bae0dd7723d775c485a Summary: There's little of interest that can be done to an already-erased instruction. You can't inspect it, write it to a debug log, etc. It ought to be notification that we're about to erase it. Rename the function to clarify the timing of the event and reflect current usage. Also fixed one case where we were trying to print an erased instruction. Reviewers: aditya_nandakumar Reviewed By: aditya_nandakumar Subscribers: rovka, kristof.beyls, llvm-commits Differential Revision: https://reviews.llvm.org/D55611 llvm-svn: 348976 2018-12-05T20:14:52+00:00 Aditya Nandakumar aditya_nandakumar@apple.com 2018-12-05T20:14:52+00:00 urn:sha1:f75d4f329cc45542cac4eaa375bad84e0535f278 https://reviews.llvm.org/D54980 This provides a standard API across GISel passes to observe and notify passes about changes (insertions/deletions/mutations) to MachineInstrs. This patch also removes the recordInsertion method in MachineIRBuilder and instead provides method to setObserver. Reviewed by: vkeles. llvm-svn: 348406