[PATCH] D21743: Convenience function for MCContext to get named sections when lowering.

Rafael Ávila de Espíndola via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 29 18:34:26 PDT 2016


rafael added a comment.

Much better. It should at least rename Section and Group to just Prefix and Suffix, since there nothing specific about groups.

Having mentioned that, would it be too expensive to make the existing getELFSection overloads take a Twine instead of a StringRef?


http://reviews.llvm.org/D21743





More information about the llvm-commits mailing list