thesamesam added a comment.

In D159115#4638325 <https://reviews.llvm.org/D159115#4638325>, @v.g.vassilev 
wrote:

> In D159115#4638323 <https://reviews.llvm.org/D159115#4638323>, @mgorny wrote:
>
>> Changing the type from `unsigned long long` to `uintptr_t` fix the test for 
>> me.
>
> Ah! Nice catch! Can you commit the fix?

Done, thanks to @mgorny for the testing and thank you @v.g.vassilev for the 
quick ack!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D159115

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
  • [PATCH] D... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Lang Hames via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits
    • [PAT... Sam James via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Michał Górny via Phabricator via cfe-commits
    • [PAT... Tulio Magno Quites Machado Filho via Phabricator via cfe-commits
    • [PAT... Vassil Vassilev via Phabricator via cfe-commits

Reply via email to