This is an archive of the discontinued LLVM Phabricator instance.

[X86] Move back _mulx_u32 to 32-bit only
Needs ReviewPublic

Authored by pengfei on Jun 23 2023, 8:17 PM.

Details

Summary

We cannot lower it to mulx at the present due to backend reason.
Since GCC doesn't support it on 64-bit, it's OK to leave it 32-bit only.

Diff Detail

Event Timeline

pengfei created this revision.Jun 23 2023, 8:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 23 2023, 8:17 PM
pengfei requested review of this revision.Jun 23 2023, 8:17 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 23 2023, 8:17 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript

I'm okay with putting this back if the codegen (D153620) can't be resolved. Honestly I didn't think it was a problem.
But I think Craig or Simon should sign off.