From 91dfdd5734e1c0a603e17bae0dd7723d775c485a Mon Sep 17 00:00:00 2001 From: Daniel Sanders Date: Wed, 12 Dec 2018 21:32:01 +0000 Subject: [globalisel] Rename GISelChangeObserver's erasedInstr() to erasingInstr() and related nits. NFC 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 --- llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp') diff --git a/llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp b/llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp index 997bd926305..56cf45e07d3 100644 --- a/llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp +++ b/llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp @@ -15,7 +15,7 @@ class DummyGISelObserver : public GISelChangeObserver { public: void changedInstr(MachineInstr &MI) override {} void createdInstr(MachineInstr &MI) override {} - void erasedInstr(MachineInstr &MI) override {} + void erasingInstr(MachineInstr &MI) override {} }; // Test CTTZ expansion when CTTZ_ZERO_UNDEF is legal or custom, -- cgit v1.2.3