[clang] [llvm] [clang][OpenMP][SPIR-V] Fix AS of globals and set the default AS to 4 (PR #135251)
Alex Voicu via cfe-commits
cfe-commits at lists.llvm.org
Thu Apr 24 06:10:46 PDT 2025
AlexVlx wrote:
Overall this seems a bit AS cast heavy in the OMP parts, I wonder if we have a chance to figure out if we cannot just emit things in the right ASes from the get-go? This is the path we took when cleaning up some of this stuff in Clang for C/C++, and whilst it was definitely more painful than casting locally for the person authoring the patch, it ended up being somewhat tidier/more robust. Otherwise, I have the same thought from the CUDA-fix PR, which is it might be worthwhile to "undummify" this AS map altogether.
https://github.com/llvm/llvm-project/pull/135251
More information about the cfe-commits
mailing list