rnk accepted this revision.
rnk added a comment.
This revision is now accepted and ready to land.
Looks good, and I did ask Richard about this in person before filing the bug
and he was in favor of it, so feel free to commit. If I'd known how easy it was
to implement and how few tests it would break, I would've made the patch
instead of filing it. :) Thanks for getting around to it.
================
Comment at: test/CodeGen/ubsan-volatile.c:9
+ // CHECK-NEXT: ret void
+ volatile int *p;
+ *p;
----------------
IMO the CHECKs would be cleaner if this were a parameter. You can do
`CHECK-NOT: call{{.*}}ubsan` or something.
https://reviews.llvm.org/D34262
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
