[llvm-branch-commits] [llvm] [NFC][KeyInstr] Add Atom Group (re)mapping (PR #133479)

Jeremy Morse via llvm-branch-commits llvm-branch-commits at lists.llvm.org
Thu Apr 10 08:05:08 PDT 2025


================
@@ -105,6 +105,13 @@ enum RemapFlags {
   /// Any global values not in value map are mapped to null instead of mapping
   /// to self.  Illegal if RF_IgnoreMissingLocals is also set.
   RF_NullMapMissingGlobalValues = 8,
+
+  /// Do not remap atom instances. Only safe if to do this if the cloned
+  /// instructions being remapped are inserted into a new function, or an
+  /// existing function where the inlined-at fields are updated. If in doubt,
+  /// don't use this flag. It's used for compiler performance reasons rather
+  /// than correctness.
----------------
jmorse wrote:

```suggestion
  /// don't use this flag. It's used when remapping is known to be un-necessary
  /// to save some compile-time.
```

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


More information about the llvm-branch-commits mailing list