[PATCH] D28476: [Analysis] Add LF_ prefix to LibFunc enums in TargetLibraryInfo. (NFC)

Richard Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 11:16:39 PST 2017


rsmith added a comment.

This makes sense to me; the current names seem pretty much optimally chosen to collide with C library functions (particularly ones that are not required to be `#undef`'d by a conforming implementation), as they are names we *know* that some C libraries provide as a non-conforming extension. Eli, can you make your peace with this, or do you still prefer that we `#undef` these names?


https://reviews.llvm.org/D28476





More information about the llvm-commits mailing list