Displaying 2 results from an estimated 2 matches for "strippointercastssamerepresentation".
2020 Mar 23
3
[InstCombine] Addrspacecast and GEP assumed commutative
...onCombining.cpp#L2175):
// Handle gep(bitcast x) and gep(gep x, 0, 0, 0).
Value *StrippedPtr = PtrOp->stripPointerCasts();
PointerType *StrippedPtrTy = cast<PointerType>(StrippedPtr->getType());
Namely, the stripPointerCasts function also strips address spaces.
Would replacing this by stripPointerCastsSameRepresentation be an acceptable solution, or do other parts of LLVM rely on this reordering?
If not, I can send a patch in Phabricator to address this issue.
Kind regards,
Thomas Faingnaert
________________________________
Van: Johannes Doerfert
Verzonden: Zaterdag, 14 Maart, 2020 6:50
Aan: Thomas Faingnaert
CC...
2020 Mar 24
2
[InstCombine] Addrspacecast and GEP assumed commutative
...ists.llvm.org <llvm-dev at lists.llvm.org>
Onderwerp: Re: [llvm-dev] [InstCombine] Addrspacecast and GEP assumed commutative
On Mar 23, 2020, at 10:52, Thomas Faingnaert via llvm-dev <llvm-dev at lists.llvm.org<mailto:llvm-dev at lists.llvm.org>> wrote:
Would replacing this by stripPointerCastsSameRepresentation be an acceptable solution, or do other parts of LLVM rely on this reordering?
If not, I can send a patch in Phabricator to address this issue.
This is probably the right solution. Nothing should be relying on this anyway
-Matt
-------------- next part --------------
An HTML attachment was scrubbe...