Add tests to illustrate the generated IR.
Thank you for working on this, I think it's very useful functionality! I'd appreciate seeing some unit test coverage for these changes.
- Remove VIACopy changes and fix formating
Fix clang-tidy issues. This cause inconsistency in code style in asan (snake case vs camel case), because I did not want to touch all the asan code in this patch.
Rebase and ping.
Address review comments.
It seems like the macro's in StmtVisitor.h only work if CAO and BinaryOperator are separate classes.
Address some of the review comments.
Adding a testcase would be tricky as these register overlap in memory and we store them with overlapping offsets with their children we should not need to invalidate the children when we write the parent but for some strange unexplainable reason QEMU was behaving strangely and not updating the first half in certain random cases. I just thought invalidation of children will force a read after write for that case.
Looks like this broke some buildbots: