Developers of the FFmpeg multimedia package raised concerns and asked AMD to be more cautious when submitting patches, especially those generated through AI, without manual review. The dissatisfaction stemmed from a set of patches aimed at implementing the use of AMD HIP SDK on the Windows platform for accelerating video processing on AMD GPU systems.
The patches in question were criticized for containing meaningless code and unnecessary documentation. For example, a constant named “EIGHT” with a value of 8 (const int EIGHT = 8) and unnecessary instructions for installing gcc and make using the pacman package manager. It was suspected that these changes were AI-generated and went unnoticed due to inadequate review.
The patch author defended the inclusion of pacman instructions, stating that they were added deliberately as he believed they would be helpful for installing dependencies. However, he expressed willingness to remove them if deemed unnecessary by the reviewer.
In response, a representative from FFmpeg apologized for the confusion and highlighted the importance of clear and concise commit messages explaining the purpose of the changes. He emphasized that important details like setting up the environment should not be solely relied upon from the commit note.