[DebugInfo][InstrRef][NFC] Add a missing assignment operator
authorJeremy Morse <jmorse+git@studentrobotics.org>
Sun, 30 Jan 2022 20:25:00 +0000 (20:25 +0000)
committerJeremy Morse <jeremy.morse@sony.com>
Mon, 31 Jan 2022 16:08:38 +0000 (16:08 +0000)
ValueIDNum is supposed to be a value type that boils down to a uint64_t,
that has some bitfields for convenience. If we use the default operator=,
we end up with each bit field being individually assigned, which is
un-necessarily slow.

Implement the assignment operator by just copying the uint64_t value of
the object. This is quicker, and matches how the comparison operators
work already. Doing so is 0.1% faster on the compile-time-tracker.

llvm/lib/CodeGen/LiveDebugValues/InstrRefBasedImpl.h

index ed0c55a..6c0d14e 100644 (file)
@@ -132,6 +132,11 @@ public:
     u.s = {Block, Inst, Loc.asU64()};
   }
 
+  ValueIDNum &operator=(const ValueIDNum &Other) {
+    u.Value = Other.u.Value;
+    return *this;
+  }
+
   uint64_t getBlock() const { return u.s.BlockNo; }
   uint64_t getInst() const { return u.s.InstNo; }
   uint64_t getLoc() const { return u.s.LocNo; }