Page MenuHomePhabricator

[opaque pointer types] GEP: replace getPointerElementType uses with get{Source,Result}ElementType.
AbandonedPublic

Authored by eddyb on Jan 16 2016, 4:59 PM.

Details

Reviewers
mjacob
Summary

[breaking-change] SimplifyGEPInst: take the source element type of the GEP as an argument.

[breaking-change] gep_type_{begin,end} now take source element type and address space.

[breaking-change] DataLayout::getIndexedOffset: take source element type instead of pointer type.

GEPOperator: provide getResultElementType alongside getSourceElementType.

ConstantFoldInstOperands: add a form taking the Instruction or ConstantExpr for GEPs.

GEP: replace getPointerElementType uses with get{Source,Result}ElementType.

Diff Detail

Event Timeline

eddyb updated this revision to Diff 45086.Jan 16 2016, 4:59 PM
eddyb retitled this revision from to [opaque pointer types] GEP: replace getPointerElementType uses with get{Source,Result}ElementType..
eddyb updated this object.
eddyb added a reviewer: mjacob.
eddyb added subscribers: dblaikie, llvm-commits.
eddyb abandoned this revision.Jan 17 2016, 2:56 PM

I've split this up as well, only 2 reviews left are http://reviews.llvm.org/D16275 and http://reviews.llvm.org/D16282.