[PATCH] D38419: Rename instruction classes for memory intrinsics. (NFC)

Hal Finkel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 29 12:57:29 PDT 2017


hfinkel added a comment.

In https://reviews.llvm.org/D38419#884799, @dneilson wrote:

> Pulling this, for now. I'm going to make sure that the joint part of the hierarchy (i.e. the AnyMem* classes) will work before trying to land this part.


FWIW, I don't really like the PainMem* names, although the AtomicMem* looks like an improvement. Can't you just leave the plain ones alone?


https://reviews.llvm.org/D38419





More information about the llvm-commits mailing list