This is an archive of the discontinued LLVM Phabricator instance.

[NVPTX] Fixed param symbol name lowering in unnamed funcs. Fixes PR40817
AbandonedPublic

Authored by jascha on Feb 22 2019, 2:06 AM.

Details

Summary

NVPTXTargetLowering::getParamSymbol used MachineFunction::getName
instead of the mangled symbol name. For unnamed functions (e.g. like
IPO/MergeFunction may create them) this produced an operand name that is
inconsistent with other parts of this module that emit parameter names.

Diff Detail

Event Timeline

jascha created this revision.Feb 22 2019, 2:06 AM
Herald added a project: Restricted Project. · View Herald TranscriptFeb 22 2019, 2:06 AM
tra added a comment.Feb 22 2019, 9:35 AM

Could you add a test to demonstrate the issue and verify that it is fixed?

jascha abandoned this revision.May 2 2020, 7:14 AM

Patch has been added sometime before c8238de61fc2a.