[globalisel] Rename GISelChangeObserver's erasedInstr() to erasingInstr() and related...
authorDaniel Sanders <daniel_l_sanders@apple.com>
Wed, 12 Dec 2018 21:32:01 +0000 (21:32 +0000)
committerDaniel Sanders <daniel_l_sanders@apple.com>
Wed, 12 Dec 2018 21:32:01 +0000 (21:32 +0000)
commit91dfdd5734e1c0a603e17bae0dd7723d775c485a
tree6aa0700733f665b7b1701c5f69e87f4fec547d7a
parentd1c61861ddc94457b08a5a653d3908b7b38ebb22
[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/include/llvm/CodeGen/GlobalISel/CombinerHelper.h
llvm/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h
llvm/lib/CodeGen/GlobalISel/Combiner.cpp
llvm/lib/CodeGen/GlobalISel/CombinerHelper.cpp
llvm/lib/CodeGen/GlobalISel/Legalizer.cpp
llvm/unittests/CodeGen/GlobalISel/LegalizerHelperTest.cpp