[PATCH] D91323: [PowerPC] Add support for intrinsics dcbfps and dcbstps in P10.

EsmeYi via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Nov 24 22:46:15 PST 2020


Esme added a comment.

@steven.zhang I still have some doubts about the properties setting. According to the behavior described by ISA, it also includes the write operation?

> ...those locations are **written** to main storage and additional locations in the block may be **written** to main storage...

I think it would be a safer way to only set `IntrArgMemOnly`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D91323



More information about the llvm-commits mailing list