[PATCH] D148906: [AMDGPU] Remove function if FeatureWavefrontSize 32 is not supported on current GPU

[Jay Foad via Phabricator via llvm-commits]

foad added a comment.

> I think we need to similarly remove a wavefrontsize64 function if compiling in wave32 mode.

I think this is a reasonable request but it's an extension to what this pass currently does.

- Currently the pass removes functions that rely on features not supported by the `-mcpu=` GPU.
- The extension would be to remove functions that rely on features not supported by the `-mcpu=` GPU **as modified by any `-mattr=` options**.

Does that make sense? If so can we please implement that in a generic way instead of having a special hack for wave32/wave4 features?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D148906/new/

https://reviews.llvm.org/D148906