[llvm] r326810 - [Asm] Fix another layering violation in assmebly macro dumping

Pavel Labath via llvm-commits llvm-commits at lists.llvm.org
Thu Mar 15 02:49:01 PDT 2018


The people who reviewed the original patch would probably be a good place
to start. You can also look at CODE_OWNERS.txt and/or commit history for
inspiration.

On Thu, 15 Mar 2018 at 09:42, Roger Pau Monné <royger at freebsd.org> wrote:

> On Wed, Mar 14, 2018 at 02:50:21PM +0000, David Blaikie wrote:
> > Possibly - you probably want to pull in some folks who own/are familiar
> > with that part of the code base to see if that's appropriate layering
> here?
> > (it seems slightly strange to talk about tokens outside the context of a
> > parser to me - but I guess there's need for it)
>
> Do you have any names of people I should pull in? I'm not a clang
> developer, this is a one-time fix I did because it was needed to
> compile another project, so I'm a little bit lost about who I should
> ask.
>
> I can create a review with the proposed fix and add them.
>
> Thanks, Roger.
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20180315/916a2801/attachment.html>


More information about the llvm-commits mailing list