This is an archive of the discontinued LLVM Phabricator instance.

CodeGen: Use MachineInstr& in TargetLowering, NFC
ClosedPublic

Authored by dexonsmith on Jun 28 2016, 9:50 PM.

Details

Summary

This is a mechanical change to make TargetLowering API take MachineInstr&
(instead of MachineInstr*), since the argument is expected to be a valid
MachineInstr. In one case, changed a parameter from MachineInstr* to
MachineBasicBlock::iterator, since it was used as an insertion point.

As a side effect, this removes a bunch of MachineInstr* to
MachineBasicBlock::iterator implicit conversions, a necessary step
toward fixing PR26753.

Diff Detail

Event Timeline

dexonsmith updated this revision to Diff 62176.Jun 28 2016, 9:50 PM
dexonsmith retitled this revision from to CodeGen: Use MachineInstr& in TargetLowering, NFC .
dexonsmith updated this object.
dexonsmith added a subscriber: llvm-commits.
ab accepted this revision.Jun 29 2016, 6:09 AM
ab added a reviewer: ab.
ab added subscribers: dylanmckay, ab.

Looks straightforward enough, go for it!

lib/Target/AVR/AVRISelLowering.h
95–97

This isn't defined anywhere, but it looks like Dylan is working on completing r268722.

This revision is now accepted and ready to land.Jun 29 2016, 6:09 AM

This was r274287, earlier today. (Sorry I forgot to report on this thread until now.)

dexonsmith closed this revision.Jun 30 2016, 7:04 PM

Fixed in r274287.