[PATCH] D74621: ValueTracking: Fix crash on weird size bswap

Nikita Popov via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Feb 16 13:33:45 PST 2020


nikic added a comment.

So the APInt support is there now, but we still have to do something here. I would suggest to enforce this requirement in the IR verifier (it already has a bunch of similar checks in visitIntrinsicCall), so that individual code handling bswaps doesn't have to deal with it.


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

https://reviews.llvm.org/D74621





More information about the llvm-commits mailing list