This is an archive of the discontinued LLVM Phabricator instance.

AMDGPU: Introduce a flag to disable mul24 intrinsic formation
ClosedPublic

Authored by arsenm on Aug 24 2019, 1:55 PM.

Details

Diff Detail

Event Timeline

arsenm created this revision.Aug 24 2019, 1:55 PM
This revision is now accepted and ready to land.Aug 24 2019, 2:04 PM
arsenm closed this revision.Aug 24 2019, 3:13 PM

r369856

What's the reason for having this flag?

arsenm added a comment.Oct 2 2019, 7:55 AM

What's the reason for having this flag?

There’s an open bug alleging the original change introduced a regression

Huh, interesting. I guess nobody had the time to really dig into why. Anyway, thanks. I think it'd be good to put this kind of information into our commit messages.