[PATCH] D104550: [analyzer] Implement getType for SVal

Wed Jun 30 03:06:37 PDT 2021

vsavchenko added a comment.

In D104550#2849561 <https://reviews.llvm.org/D104550#2849561>, @DavidSpickett wrote:

> @vsavchenko One of the added tests is failing on our 32 bit Armv7 Thumb bot: https://lab.llvm.org/buildbot/#/builders/170/builds/61
>
>   /home/tcwg-buildslave/worker/clang-thumbv7-full-2stage/llvm/clang/unittests/StaticAnalyzer/SValTest.cpp:169: Failure
>   Expected equality of these values:
>     Context.UnsignedLongTy
>       Which is: unsigned long
>     A.getType(Context)
>       Which is: unsigned int
>   [  FAILED  ] SValTest.GetLocAsIntType (22 ms)
>   [----------] 1 test from SValTest (22 ms total)
>
> A 32/64 bit issue?

Hi @DavidSpickett , thanks for looking into this!
This patch was almost instantly followed by https://github.com/llvm/llvm-project/commit/b2842298cebf420ecb3750bf309021a7f37870c1 which fixed the issue.  Please, let me know, if you still see it after that commit!

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D104550