This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Don't consider whether amdgpu-flat-work-group-size was set
ClosedPublic

Authored by arsenm on Sep 13 2021, 9:52 AM.

Details

Summary

It should be semantically identical if it was set to the same value as
the default.

Diff Detail

Event Timeline

arsenm created this revision.Sep 13 2021, 9:52 AM
arsenm requested review of this revision.Sep 13 2021, 9:52 AM
Herald added a project: Restricted Project. · View Herald TranscriptSep 13 2021, 9:52 AM
Herald added a subscriber: wdng. · View Herald Transcript

need a test to show the effect of this change.

arsenm updated this revision to Diff 373311.Sep 17 2021, 12:37 PM

Missed test updates

Is it the same as D109704?

llvm/docs/AMDGPUUsage.rst
872

Reformat the column.

arsenm updated this revision to Diff 373729.Sep 20 2021, 3:40 PM

Indentation

Indentation

Test is still wider than the column.

Indentation

Test is still wider than the column.

lit tests have never strictly followed the column rule and we don't have tooling to automatically line wrap them

Indentation

Test is still wider than the column.

lit tests have never strictly followed the column rule and we don't have tooling to automatically line wrap them

Sorry, should read "text is still wider than 'Description' column.

arsenm updated this revision to Diff 381641.Oct 22 2021, 12:54 PM

Reformat. None of the other columns follow the header here, but it doesn't really matter since the output is reformatted anyway

This revision is now accepted and ready to land.Oct 22 2021, 12:58 PM