[Openmp-commits] [openmp] [OpenMP][AMDGPU] Adapt dynamic callstack sizes to HIP behavior (PR #74080)

Joseph Huber via Openmp-commits openmp-commits at lists.llvm.org
Mon Dec 4 11:22:43 PST 2023

jhuber6 wrote:

> > Jay's suggestion sounds good to me. HIP has different tradeoffs from living in a different repo - maybe the right thing to do there is include the header in comgr and re-export the function, but it's all very heavy handed relative to the #include and done approach in tree runtimes get to use.
> On my first attempt to rename & move the corresponding `Generation` enum into `lib/Target/AMDGPU/Utils/AMDGPUBaseInfo.h` I would need to adapt (at least) ~ 25 other files within the backend; and TBH I've yet to get it to work as expected.
> @jayfoad @JonChesterfield @jhuber6 Q: How should I go about this change to avoid introducing (lingering?) errors into the backend? Any suggestions welcome! (Is there a way to narrow that number? [I guess not.])

Can you just make an inline function somewhere that the current implementation also calls?


More information about the Openmp-commits mailing list