[PATCH] D152899: Introduce RThreadAccess pass

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jul 11 12:36:22 PDT 2023


arsenm added a comment.

In D152899#4470137 <https://reviews.llvm.org/D152899#4470137>, @yrouban wrote:

> In D152899#4447418 <https://reviews.llvm.org/D152899#4447418>, @arsenm wrote:
>
>> Right, this is the ugly thing I'm trying to guide you away from doing but need to understand exactly what the constraints are on the register. An intrinsic with a defined meaning is better than an unrestricted register read.
>
> Thank you. An excuse for reading register is that it is done in the very late stage of the pipeline and is almost target agnostic.

I find the areas where people try to bypass the normal legalization process difficult to follow and implement. I think expecting to parse meaning out of an unrestricted read and a side channel to try to interpret it is more difficult to deal with rather than something higher level


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D152899/new/

https://reviews.llvm.org/D152899



More information about the llvm-commits mailing list