This is an archive of the discontinued LLVM Phabricator instance.

[MLIR] Add support for alloca() of non-zero address spaces
ClosedPublic

Authored by krzysz00 on Feb 14 2022, 1:25 PM.

Details

Summary

This change is needed when lowering alloca()-using code on targets
such as ROCDL that represent private scratch space as a separate
address space.

Diff Detail

Event Timeline

krzysz00 created this revision.Feb 14 2022, 1:25 PM
krzysz00 requested review of this revision.Feb 14 2022, 1:25 PM

Can this be reflected in a test?

ftynse requested changes to this revision.Feb 15 2022, 1:10 AM

Please add a test for this, looks good otherwise.

This revision now requires changes to proceed.Feb 15 2022, 1:10 AM
krzysz00 updated this revision to Diff 408902.Feb 15 2022, 8:58 AM
  • Add a test for nonzero address space on llvm.alloca
ftynse accepted this revision.Feb 15 2022, 8:59 AM
This revision is now accepted and ready to land.Feb 15 2022, 8:59 AM
krzysz00 updated this revision to Diff 408910.Feb 15 2022, 9:13 AM

Rebase so there's a working build