<html>

<head>

<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">

<style type="text/css" style="display:none;"> P {margin-top:0;margin-bottom:0;} </style>

</head>

<body dir="ltr">

Hi Alexey,

<div><br>

</div>

<div>> John, Anastasia, does <a href="https://reviews.llvm.org/D62574" title="https://reviews.llvm.org/D62574">

https://reviews.llvm.org/D62574</a> seem reasonable to you? </div>

<div>If so, I can rebase my patch of top of this one. </div>

<div><br>

</div>

<div>I have just reviewed the patch again and it generally looks fine. </div>

<div><br>

</div>

<div>However, I guess it will still not solve the address space deduction that you

</div>

<div>discussed earlier? It also doesn't help to eliminate address spaces from AST by</div>

<div>converting them to generic that is what you need too? I still think that the main</div>

<div>concept of the address space attribute in Clang is the ability to propagate and</div>

<div>preserve different address spaces in types of AST, should they occur explicitly in</div>

<div>the source, and provide checks/diagnostics for mismatching address spaces.</div>

<div>But if this logic is undesirable for you I would encourage you to look at different

</div>

<div>attributes, perhaps even at adding a new one. After all, you don't need to have</div>

<div>address spaces in AST types in order to propagate them to IR. </div>

<div><br>

</div>

<div>Cheers, </div>

Anastasia<br>

<div id="appendonsend"></div>

<div style="font-family:Calibri,Arial,Helvetica,sans-serif; font-size:12pt; color:rgb(0,0,0)">

<br>

</div>

<hr tabindex="-1" style="display:inline-block; width:98%">

<div id="divRplyFwdMsg" dir="ltr"><font style="font-size:11pt" face="Calibri, sans-serif" color="#000000"><b>From:</b> Bader, Alexey <alexey.bader@intel.com><br>

<b>Sent:</b> 14 August 2020 22:31<br>

<b>To:</b> Bevin Hansson <bevin.hansson@ericsson.com>; 'John McCall' <rjmccall@apple.com>; via cfe-dev <cfe-dev@lists.llvm.org>; Anastasia Stulova <Anastasia.Stulova@arm.com><br>

<b>Cc:</b> nd <nd@arm.com>; Victor Lomuller <victor@codeplay.com><br>

<b>Subject:</b> RE: [cfe-dev] [RFC] Re-use OpenCL address space attributes for SYCL</font>

<div> </div>

</div>

<div class="BodyFragment"><font size="2"><span style="font-size:11pt">

<div class="PlainText">Hi Bevin,<br>

<br>

Thank you for the head-up.<br>

If we can commit this patch soon, we can easily limit conversion between default and OpenCL address spaces to SYCL mode to avoid any interference with OpenCL mode.<br>

I think it's more preferable to do for the language mode to avoid overloading TargetInfo methods for all supported targets (currently SPIR and NVTPX).<br>

<br>

John, Anastasia, does <a href="https://reviews.llvm.org/D62574">https://reviews.llvm.org/D62574</a> seem reasonable to you? If so, I can rebase my patch of top of this one.<br>

<br>

Particularly this change makes it easier to define different behavior for different language modes:<br>

"Moves QualType/Qualifiers accessors which deal with qualifier relations (such as compatiblyIncludes, etc.) to ASTContext, as Qualifiers cannot be made aware of the relations between address spaces on their own."<br>

<br>

Thanks,<br>

Alexey <br>

<br>

-----Original Message-----<br>

From: Bevin Hansson <bevin.hansson@ericsson.com> <br>

Sent: Thursday, August 6, 2020 3:22 PM<br>

To: 'John McCall' <rjmccall@apple.com>; Bader, Alexey <alexey.bader@intel.com>; via cfe-dev <cfe-dev@lists.llvm.org><br>

Cc: nd <nd@arm.com><br>

Subject: RE: [cfe-dev] [RFC] Re-use OpenCL address space attributes for SYCL<br>

<br>

Hi,<br>

<br>

>Treating the default address space as a superspace of some (all?) of the other address spaces is sensible and should definitely work.<br>

>There are already predicates in the AST for deciding whether two address spaces are related this way; they probably need to be target-customizable, but that should be straightforward.<br>

<br>

Just to chime in here; I have a patch for making address space relations target-configurable here:

<a href="https://reviews.llvm.org/D62574">https://reviews.llvm.org/D62574</a> <br>

<br>

/ Bevin<br>

<br>

_______________________________________________<br>

cfe-dev mailing list<br>

cfe-dev@lists.llvm.org<br>

<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev">https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-dev</a><br>

</div>

</span></font></div>

</body>

</html>