This is an archive of the discontinued LLVM Phabricator instance.

[AMDGPU][NFC] Fix typos
ClosedPublic

Authored by sebastian-ne on Nov 11 2021, 6:34 AM.

Diff Detail

Event Timeline

sebastian-ne created this revision.Nov 11 2021, 6:34 AM
sebastian-ne requested review of this revision.Nov 11 2021, 6:34 AM
Herald added a project: Restricted Project. · View Herald TranscriptNov 11 2021, 6:34 AM
foad added inline comments.Nov 11 2021, 6:43 AM
llvm/lib/Target/AMDGPU/AMDGPULegalizerInfo.h
112

If you're changing this, maybe use "Den" too like in the function above?

llvm/lib/Target/AMDGPU/AMDGPULibFunc.cpp
780

I can't see a typo fix here. Is it just line wrapping?

sebastian-ne marked an inline comment as done.

Abbreviate Denom to Den

llvm/lib/Target/AMDGPU/AMDGPULibFunc.cpp
780

Yes, just line wrapping from running clang-format afterwards.

foad accepted this revision.Nov 11 2021, 7:22 AM
This revision is now accepted and ready to land.Nov 11 2021, 7:22 AM
This revision was automatically updated to reflect the committed changes.