Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(47)

Issue 4769041: [PATCH] Add APInt::getSigned(bits, value)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 9 months ago by Jeffrey Yasskin
Modified:
12 years, 9 months ago
Reviewers:
Visibility:
Public.

Description

Add APInt::getSigned(bits, value) to avoid ambiguity between the APInt(bits, uint64_t, bool) and APInt(bits, unsigned, uint64_t*) constructors. Deprecate both of those constructors, and update LLVM's calls to them.

Patch Set 1 #

Patch Set 2 : Rebase #

Patch Set 3 : Fixes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+61 lines, -34 lines) Patch
M docs/ReleaseNotes.html View 1 2 2 chunks +5 lines, -0 lines 0 comments Download
M include/llvm/ADT/APInt.h View 1 4 chunks +19 lines, -5 lines 0 comments Download
M lib/ExecutionEngine/ExecutionEngineBindings.cpp View 1 chunk +5 lines, -1 line 0 comments Download
M lib/Support/APInt.cpp View 1 2 1 chunk +1 line, -1 line 0 comments Download
M lib/Target/CellSPU/SPUISelLowering.cpp View 1 chunk +1 line, -1 line 0 comments Download
M lib/Transforms/InstCombine/InstCombineMulDivRem.cpp View 1 chunk +3 lines, -2 lines 0 comments Download
M lib/Transforms/Scalar/LoopStrengthReduce.cpp View 1 chunk +1 line, -1 line 0 comments Download
M lib/VMCore/Constants.cpp View 1 chunk +4 lines, -1 line 0 comments Download
M unittests/ADT/APFloatTest.cpp View 1 chunk +1 line, -1 line 0 comments Download
M unittests/ADT/APIntTest.cpp View 1 8 chunks +17 lines, -17 lines 0 comments Download
M unittests/Support/ConstantRangeTest.cpp View 1 chunk +4 lines, -4 lines 0 comments Download

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b