[llvm] r353741 - [globalisel] Restore comment explaining the nits of GISelChangeObserver::createdInstr()

Daniel Sanders via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 11 12:05:49 PST 2019


Author: dsanders
Date: Mon Feb 11 12:05:49 2019
New Revision: 353741

URL: http://llvm.org/viewvc/llvm-project?rev=353741&view=rev
Log:
[globalisel] Restore comment explaining the nits of GISelChangeObserver::createdInstr()


Modified:
    llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h

Modified: llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h?rev=353741&r1=353740&r2=353741&view=diff
==============================================================================
--- llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h (original)
+++ llvm/trunk/include/llvm/CodeGen/GlobalISel/GISelChangeObserver.h Mon Feb 11 12:05:49 2019
@@ -33,10 +33,17 @@ public:
 
   /// An instruction is about to be erased.
   virtual void erasingInstr(MachineInstr &MI) = 0;
-  /// An instruction was created and inserted into the function.
+
+  /// An instruction has been created and inserted into the function.
+  /// Note that the instruction might not be a fully fledged instruction at this
+  /// point and won't be if the MachineFunction::Delegate is calling it. This is
+  /// because the delegate only sees the construction of the MachineInstr before
+  /// operands have been added.
   virtual void createdInstr(MachineInstr &MI) = 0;
+
   /// This instruction is about to be mutated in some way.
   virtual void changingInstr(MachineInstr &MI) = 0;
+
   /// This instruction was mutated in some way.
   virtual void changedInstr(MachineInstr &MI) = 0;
 




More information about the llvm-commits mailing list