[PATCH] D72741: [MLIR] LLVM dialect: Add llvm.atomicrmw

Frank Laub via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 17 11:21:04 PST 2020


flaub added a comment.

I'm happy to do whatever the community wants to do. I was under the impression from @ftynse that we shouldn't be changing the MLIR IR, that it should match exactly whatever the LLVM IR is. It's unclear to me what the ramifications of making such a change in LLVM IR are, so I don't feel qualified at this time to make such a change. But, I could certainly take some guidance on this. (this is where my pushback is coming from, sorry if this came off the wrong way).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D72741





More information about the llvm-commits mailing list