[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