diff options
| author | James Y Knight <jyknight@google.com> | 2019-02-09 22:22:28 +0000 |
|---|---|---|
| committer | James Y Knight <jyknight@google.com> | 2019-02-09 22:22:28 +0000 |
| commit | 751fe286dc39227a01ecd8e6a92a4b0392fc0eb6 (patch) | |
| tree | da21e1dac258a6706074ce344dd968b47ebf97fc /clang/lib/CodeGen/CGNonTrivialStruct.cpp | |
| parent | a561d46633db845079102bb396581e9cb9c11ae7 (diff) | |
| download | bcm5719-llvm-751fe286dc39227a01ecd8e6a92a4b0392fc0eb6.tar.gz bcm5719-llvm-751fe286dc39227a01ecd8e6a92a4b0392fc0eb6.zip | |
[opaque pointer types] Cleanup CGBuilder's Create*GEP.
The various EltSize, Offset, DataLayout, and StructLayout arguments
are all computable from the Address's element type and the DataLayout
which the CGBuilder already has access to.
After having previously asserted that the computed values are the same
as those passed in, now remove the redundant arguments from
CGBuilder's Create*GEP functions.
Differential Revision: https://reviews.llvm.org/D57767
llvm-svn: 353629
Diffstat (limited to 'clang/lib/CodeGen/CGNonTrivialStruct.cpp')
| -rw-r--r-- | clang/lib/CodeGen/CGNonTrivialStruct.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/clang/lib/CodeGen/CGNonTrivialStruct.cpp b/clang/lib/CodeGen/CGNonTrivialStruct.cpp index f7fe62bc4b0..a1649e4fb7b 100644 --- a/clang/lib/CodeGen/CGNonTrivialStruct.cpp +++ b/clang/lib/CodeGen/CGNonTrivialStruct.cpp @@ -413,8 +413,7 @@ template <class Derived> struct GenFuncBase { if (Offset.getQuantity() == 0) return Addr; Addr = CGF->Builder.CreateBitCast(Addr, CGF->CGM.Int8PtrTy); - Addr = CGF->Builder.CreateConstInBoundsGEP(Addr, Offset.getQuantity(), - CharUnits::One()); + Addr = CGF->Builder.CreateConstInBoundsGEP(Addr, Offset.getQuantity()); return CGF->Builder.CreateBitCast(Addr, CGF->CGM.Int8PtrPtrTy); } |
