[PATCH] D34504: [LLVM][X86][Goldmont] Adding new target-cpu: Goldmont

[Craig Topper via Phabricator via llvm-commits]

craig.topper added a comment.

Should we OR in isGLM() every place we use isSLM() today?

@zvi or @igorb Should all the places that use isAtom() also include isSLM() and isGLM()? Should isAtom() be renamed to isBonnell()? isAtom sounds like it implies any Atom processor, but it really doesn't.


https://reviews.llvm.org/D34504