[llvm] Merge sourcelocation in CSEMIRBuilder::getDominatingInstrForID. (PR #90922)

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Fri May 3 08:39:16 PDT 2024


================
@@ -47,8 +47,9 @@ class CSEMIRBuilder : public MachineIRBuilder {
   /// dominates the current insertion point and if not, move it just before the
   /// current insertion point and return it. If not found, return Null
   /// MachineInstrBuilder.
-  MachineInstrBuilder getDominatingInstrForID(FoldingSetNodeID &ID,
-                                              void *&NodeInsertPos);
+  MachineInstrBuilder
+  getDominatingInstrForID(FoldingSetNodeID &ID, void *&NodeInsertPos,
+                          SmallVectorImpl<DILocation *> *Locs = nullptr);
----------------
adrian-prantl wrote:

It would be good to document what the extra parameter is for, since this is not obvious.

https://github.com/llvm/llvm-project/pull/90922


More information about the llvm-commits mailing list