[PATCH] D109548: Make llvm.launder.invariant.group writeonly

Piotr Padlewski via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 14 01:11:01 PDT 2021


Prazek added a comment.

In D109548#2997429 <https://reviews.llvm.org/D109548#2997429>, @jdoerfert wrote:

> volatile reads are considered write effects, IIRC: https://godbolt.org/z/73Tqjj4rc

But a volatile store is considered to read and write?
https://godbolt.org/z/hjr7hrscP

This one shows that function doing volatile store is not marked with writeonly, which adds to my point that in my understanding, writeonly function with identical arguments will store the same values (which means it can be CSEd)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109548



More information about the llvm-commits mailing list