[PATCH] D39220: [Analyzer] Store BodyFarm in std::unique_ptr

Devin Coughlin via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Oct 25 13:32:40 PDT 2017


dcoughlin added a comment.

I agree with Alexander here. The LLVM naming convention is not going to change and we should err on the side of using descriptive names. How about "FunctionBodyFarm"?


Repository:
  rL LLVM

https://reviews.llvm.org/D39220





More information about the cfe-commits mailing list