This is an archive of the discontinued LLVM Phabricator instance.

[NFC][NVPTX] Remove dead comment and commented code
ClosedPublic

Authored by ldrumm on Dec 20 2022, 7:01 AM.

Details

Summary

A confusing comment after the last return statement in
NVPTXAsmPrinter::doFinalization referred to a preprocessor macro
(NVISA) that has never existed since the NVPTX backend has been a part
of upstream llvm - as far as the pickaxe will tell me anyway. Thus I've
removed it.

Diff Detail

Event Timeline

ldrumm created this revision.Dec 20 2022, 7:01 AM
ldrumm requested review of this revision.Dec 20 2022, 7:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 20 2022, 7:01 AM
This revision was not accepted when it landed; it landed in state Needs Review.Dec 23 2022, 4:11 AM
This revision was automatically updated to reflect the committed changes.