[PATCH] D105457: [clang] Refactor AST printing tests to share more infrastructure

Tue Jul 13 23:56:07 PDT 2021

dyung added a comment.

In D105457#2875672 <https://reviews.llvm.org/D105457#2875672>, @dblaikie wrote:

> In D105457#2874783 <https://reviews.llvm.org/D105457#2874783>, @nridge wrote:
>
>> To be honest, I don't really understand this error. It seems to come from a compile step (not a link step), and it comes from the assembler and says "symbol ... is already defined"? I don't think I've seen an error like that before. (Typically, errors about duplicate definitions would come from the linker.)
>>
>> Any suggestions for what it might indicate or how to investigate are welcome.
>
> Yeah, I'm with you there.
>
> @dyung @glaubitz - perhaps you two could help with a reduced example of what causes this, it does look like a bug in the compiler (yeah, I'd at first assumed a bug in the standard library that caused a link error, but as @nridge pointed out it's not a link error but an assembly error - which really looks like a compiler bug to me) - but likely there's some room for a workaround if we knew what we were dealing with and how to workaround it reliably.

@nridge, from the commit message of your revert, it sounds like you are able to locally reproduce the failure?

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105457