aaron.ballman accepted this revision.
aaron.ballman added a comment.

LGTM!



================
Comment at: clang/lib/AST/Interp/Pointer.h:63-64
 private:
   static constexpr unsigned PastEndMark = (unsigned)-1;
   static constexpr unsigned RootPtrMark = (unsigned)-1;
 
----------------
aaron.ballman wrote:
> tbaeder wrote:
> > shafik wrote:
> > > or `-1u`  
> > Alright, but those were not introduced in this patch.
> Preference for `~0u` so the types all match up nicely, fine to do in a 
> post-commit NFC change.
Don't forget to hit this one in post-commit.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D135750/new/

https://reviews.llvm.org/D135750

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D135750: [clang][Int... Aaron Ballman via Phabricator via cfe-commits

Reply via email to