[PATCH] D27031: [PM] Change the static object whose address is used to uniquely identify analyses to have a common type which is enforced rather than using a char object and a `void *` type when used as an identifier.

Chandler Carruth via llvm-commits llvm-commits at lists.llvm.org
Wed Nov 23 10:02:35 PST 2016


chandlerc added a comment.

In https://reviews.llvm.org/D27031#603946, @silvas wrote:

> This seems like an nice clarity win (even ignoring the subtle pointer alignment issues). LGTM.


Awesome, thanks for reviewing!


https://reviews.llvm.org/D27031





More information about the llvm-commits mailing list