[llvm] [AMDGPU] Make maximum hard clause size a subtarget feature (PR #81287)
Jay Foad via llvm-commits
llvm-commits at lists.llvm.org
Fri Feb 9 23:37:00 PST 2024
https://github.com/jayfoad commented:
Personally I would have gone with a new `HasClauseLengthBug` (or similar) feature, but I guess this approach is OK too.
I do think we need a brief comment (maybe on the definition of `FeatureMaxHardClauseLength32`) explaining _why_ clause length is limited to 32 on some subtargets, i.e. saying that it is a workaround for some bug rather than being something that is documented in the ISA manuals.
https://github.com/llvm/llvm-project/pull/81287
More information about the llvm-commits
mailing list