This is an archive of the discontinued LLVM Phabricator instance.

[Clang][CodeGen] Use poison instead of undef in CodeGen for ARM Builtins [NFC]
ClosedPublic

Authored by ManuelJBrito on Dec 15 2022, 2:01 AM.

Details

Summary

In light of the efforts to remove undef, this patch changes the following instances of undef to poison in the CodeGen for ARM Builtins:

  • EmitCommonNeonSISDBuiltinExpr : uses overloaded intrinsics to implement SISD expressions, ops go in first entry, the rest of the vector is irrelevant.
  • EmitAArch64SVEBuiltinExpr : cast from fixed vector to scalable vector with the same size therefore undef/poison is hidden.

Diff Detail

Event Timeline

ManuelJBrito created this revision.Dec 15 2022, 2:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 15 2022, 2:01 AM
ManuelJBrito requested review of this revision.Dec 15 2022, 2:01 AM
Herald added a project: Restricted Project. · View Herald TranscriptDec 15 2022, 2:01 AM
paulwalker-arm accepted this revision.Dec 15 2022, 3:14 AM
This revision is now accepted and ready to land.Dec 15 2022, 3:14 AM