rupprecht accepted this revision. rupprecht added a comment. This revision is now accepted and ready to land.
LGTM -- I patched this in to verify it fixes the test failure we saw. However, and not something that needs to happen in this patch (you could though), but the test actually isn't failing anymore even without this patch, because D129386 <https://reviews.llvm.org/D129386> changed the naming convention from `__cc` to `__cc_` I can get the test to correctly fail without this patch by changing the test assertion to: must_not_contain__cc = r'(?s)^(?!.*\b__cc_ = )' I'm not sure if there's a less fragile way to write that test. You could assert the entire string instead of looking for sub-fields, perhaps. Maybe that's hard to do for an unordered collection though. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133259/new/ https://reviews.llvm.org/D133259 _______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits