================
@@ -2330,14 +2330,7 @@ CIRGenModule::getAddrOfConstantStringFromLiteral(const 
StringLiteral *s,
 
 mlir::ptr::MemorySpaceAttrInterface
 CIRGenModule::getGlobalCIRAddressSpace(LangAS as) {
----------------
aobolensk wrote:

hmm... I'm not sure I get your point. There is 
CIRGenModule::getGlobalVarAddressSpace and it is untouched and all the handling 
that is similar to classic code gen you referenced is there:

CIR:

https://github.com/llvm/llvm-project/blob/d15445a80c3f67313c047f7c7d04bf8c75c646e0/clang/lib/CIR/CodeGen/CIRGenModule.cpp#L1037-L1068

vs classic:
https://github.com/llvm/llvm-project/blob/d15445a80c3f67313c047f7c7d04bf8c75c646e0/clang/lib/CodeGen/CodeGenModule.cpp#L6423-L6462

IMO, these functions are aligned here

https://github.com/llvm/llvm-project/pull/231098
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to