Anastasia added inline comments.
================ Comment at: lib/CodeGen/CGBuiltin.cpp:3711 + EventList = EventList->getType()->isIntegerTy() + ? Builder.CreateIntToPtr(EventList, EventPtrTy) + : Builder.CreatePointerCast(EventList, EventPtrTy); ---------------- AlexeySotkin wrote: > Anastasia wrote: > > It seems we are not testing the casts? > Do you mean that when we run LIT tests, this code is not executed? If so, in > the modified test below, literal zeros are making clang to execute > CreateIntToPtr call indeed. > Or, do you mean that we need some extra check(to make sure the cast will be > successful for example) in the source code itself ? I mean since you are generating extra IR nodes we should check in the tests that they appear correctly. I don't see these casts checked in the tests currently. Repository: rC Clang CHANGES SINCE LAST ACTION https://reviews.llvm.org/D59985/new/ https://reviews.llvm.org/D59985 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits