[PATCH] D83174: Teach AttachPreviousImpl to inherit MSInheritanceAttr attribute

Fri Jul 17 07:10:35 PDT 2020

aaron.ballman added reviewers: aaron.ballman, riccibruno, martong.
aaron.ballman added a comment.
Herald added a subscriber: rnkovacs.

In D83174#2156454 <https://reviews.llvm.org/D83174#2156454>, @v.g.vassilev wrote:

> >> Also, I would like to add that the current test present in this diff does not fail in the existing system but it was the best I and Vassil could come up with to replicate our problem.
> > 
> > Is there a way to run something like creduce to come up with a test case that does replicate the issue? I'd like to ensure we have a test case that fails without the patch applied so that we can be sure we don't regress the behavior.
>
> Not easily. That bug comes from internal infrastructure which uses clang's API and happens when calling directly the mangler. Running creduce would require failure to be reproducible with bare clang. I was hoping @rsmith had something up his sleeve.

Drat. While the code looks reasonable, this makes me less confident in the fix. I'm adding some more reviewers who have touched the AST serialization and modules before in case they also have ideas for test cases.

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

https://reviews.llvm.org/D83174