[PATCH] D30047: [GlobalISel] Split LowLevelType to a fundamental type in libSupport.

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 22 01:05:17 PST 2017


rovka added a comment.

In https://reviews.llvm.org/D30047#682820, @dsanders wrote:

> Ok, let's go with the standalone method. I'd like to give it a more descriptive name though seeing as it's in the llvm namespace. How about getLLTForType()?
>
> For the header name, I agree with not having LowLevelTypes.h in both libraries. Let's go with GlobalISel/Utils.h.


That sounds good to me. @ab, what do you think?


https://reviews.llvm.org/D30047





More information about the llvm-commits mailing list