This is an archive of the discontinued LLVM Phabricator instance.

IR: Remove extra name mangling from llvm.ptrmask
ClosedPublic

Authored by arsenm on May 15 2020, 10:58 AM.

Details

Summary

This should not be allowed to change the address space of the source
and result, so the pointer types should match. The only emitter of
this seems to be a disable clang change, so no tests to update.

Diff Detail

Event Timeline

arsenm created this revision.May 15 2020, 10:58 AM
Herald added a project: Restricted Project. · View Herald TranscriptMay 15 2020, 10:58 AM
Herald added a subscriber: wdng. · View Herald Transcript
fhahn accepted this revision.May 15 2020, 11:35 AM

LGTM thanks. It should indeed not change the pointer types. The LangRef already uses the same type for return value and argument, so no update should be needed there.

This revision is now accepted and ready to land.May 15 2020, 11:35 AM