[PATCH] D113934: [llvm] adapt DWARFExpression.h for 6b9b86db9dd974585a5c71cf2e5231d1e3385f7c

[Duncan P. N. Exon Smith via Phabricator via llvm-commits]

dexonsmith added a comment.

I pushed fa39ec5786c08e78cc247a48c17aef66ca5985fb, a35226b9b606f4ea32e7db9e2f4de0d99ab6bef0, and 7a1b670f0f2f0a38a673b5554bb2c4d9d50447bf to fix most of the blockers. Then https://reviews.llvm.org/D113957 fixes `Operation::verify()` and https://reviews.llvm.org/D113958 makes it iterate over `const Operation` and removes the `mutable`.

In D113934#3132954 <https://reviews.llvm.org/D113934#3132954>, @krasimir wrote:

> Sorry for submitting quickly, I just wanted to unblock build bots. I still think that this patch is OK considering the weirdness of this Operation class.

No worries; makes sense.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D113934/new/

https://reviews.llvm.org/D113934