[PATCH] D68411: Add support for intrinsics llvm.ppc.dcbfl and llvm.ppc.dcbflp

Anil Mahmud via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 28 00:00:18 PDT 2019


anil9 added a comment.

In D68411#1718116 <https://reviews.llvm.org/D68411#1718116>, @lei wrote:

> Since these are just mnemonics for using dcbf, why not just add the tests to llvm/test/CodeGen/PowerPC/dcbf.ll instead of creating 2 small test files.


These are mnemonics true, but we have separate intrinsics for them as opposed to using one intrinsic with different constants and also the assembly code is different. 
I think having multiple files gives more structure and in theory reduces chances of merge conflicts.


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

https://reviews.llvm.org/D68411





More information about the llvm-commits mailing list