[PATCH] D123508: GlobalISel: Allow forming atomic/volatile G_ZEXTLOAD
Matt Arsenault via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue May 17 14:36:16 PDT 2022
arsenm added a comment.
Herald added a subscriber: kosarev.
In D123508#3489330 <https://reviews.llvm.org/D123508#3489330>, @reames wrote:
> At a high level, I do very much think having atomic loads simply be normal loads with some atomic flags makes a lot of sense.
This is already the case for GlobalISel. I'm just extending to extending loads, rather than repeating the special case computeKnownBits hack the DAG uses (plus another poorly conceived target hook to report the atomic high bit behavior)
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D123508/new/
https://reviews.llvm.org/D123508
More information about the llvm-commits
mailing list