[PATCH] D38816: Convert clang::LangAS to a strongly typed enum

Justin Lebar via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 11 14:40:11 PDT 2017


jlebar added a comment.

> The only reason I added this namespace is that I wasn't sure whether having those functions in the clang namespace is acceptable.

Maybe someone else will object, or suggest an existing namespace they should be in.  FWIW I think it's fine.

> Not quite sure what to call the functions though. langASFromTargetAS?

sgtm!


Repository:
  rL LLVM

https://reviews.llvm.org/D38816





More information about the cfe-commits mailing list