[PATCH] D68772: [COFF] Wrap things in namespace lld { namespace coff {

Fangrui Song via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 18 07:14:00 PDT 2019


MaskRay added a comment.

In D68772#1713143 <https://reviews.llvm.org/D68772#1713143>, @rnk wrote:

> I guess the productive thing to do would be for me to put together a patch to CodingStandards and then send it to llvm-dev to get some feedback, and then follow up myself here.


Thanks! That will be very helpful. (I wanted to do the same thing but does not have enough time to do this because I am currently busy with personal (or work related) issues..) I will clean up the ELF and wasm patches (mea culpa) once the style reaches a consensus.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D68772/new/

https://reviews.llvm.org/D68772





More information about the llvm-commits mailing list