shiltian wrote: It's a good idea to make it more generic. Potentially we can add an API like `isAddressSpaceOptimizable(unsigned AS)`. If we want to do that, the question would be, where to put this API? Like I mentioned in the description, I tried some similar approach but they didn't pan out well. So far we know that, neither TTI nor `TargetMachine` is good place. Where else can we do this, other than putting it into `DataLayout`? If `DataLayout` is still a good place, then it might be just about whether we call it a _flat_ address space, or _optimizable_ address space, and nothing would be different from what is done in this PR.
https://github.com/llvm/llvm-project/pull/108786 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
