This is an archive of the discontinued LLVM Phabricator instance.

[VE] Set getExtendForAtomicOps to ISD::ANY_EXTEND
ClosedPublic

Authored by kaz7 on Jul 17 2021, 1:02 PM.

Details

Summary

The implementation of subword atomics does not actually
guarantee the result is zero-extended, which now caused
failures after https://reviews.llvm.org/D101342 was landed.

Diff Detail

Event Timeline

kaz7 created this revision.Jul 17 2021, 1:02 PM
kaz7 requested review of this revision.Jul 17 2021, 1:02 PM
kaz7 added a comment.Jul 18 2021, 9:15 AM

This fixes a problem caused in atomic_load.ll regression test.

simoll accepted this revision.Jul 19 2021, 12:03 AM
This revision is now accepted and ready to land.Jul 19 2021, 12:03 AM
This revision was landed with ongoing or failed builds.Jul 19 2021, 3:59 AM
This revision was automatically updated to reflect the committed changes.