On Fri, Jan 4, 2019 at 3:54 PM Zachary Turner <ztur...@google.com> wrote: > > It seems like we have 3 uses of this constructor in LLDB. > > IRInterpreter.cpp: Constructs a Scalar for an llvm::Constant. > IRForTarget.cpp: Constructs a Scalar for an llvm::Constant. > ClangASTContext.cpp: bitcasts an APFloat to an APInt. > > The first two we should just treat constants in LLVM IR as signed, so we > could construct an APSInt at the call-sites with signed=true. >
OK, I think there might be subtleties we're missing right now but the best way is just try and see what happens. I'll send patches early next week. -- Davide _______________________________________________ lldb-dev mailing list lldb-dev@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev