This adds a constructor and upgrades the dtor to work with
move-only types.
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
This is an obvious patch and tested with out of tree clients. There isn't an obvious owner for this file at this time besides me :-/, anyone object to landing this?
Comment Actions
LGTM - I might clarify get to be getOldValue or something to make it easier on the reader at the callsite, but I don't have strong feelings about it.
Comment Actions
agreed, I'm not in love with the existing API, but I don't want to break everyone for no reason. Thanks for the review!