[PATCH] D113934: [llvm] adapt DWARFExpression.h for 6b9b86db9dd974585a5c71cf2e5231d1e3385f7c
Duncan P. N. Exon Smith via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 15 17:43:49 PST 2021
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
More information about the llvm-commits
mailing list