Re: r213213 - DebugInfo: Forward HandleTagDeclRequiredDefinition through MultiplexConsumer to fix debug info emission in the presence of plugins.

2016-05-29 Thread David Blaikie via cfe-commits
Thanks for the catch! Fixed in r271188 On Wed, May 25, 2016 at 10:28 AM, Nico Weber wrote: > Zombie review comment: ".test" isn't part of > test/lit.cfg::config.suffixes, so the two .test files added in this change > never run unless you explicitly run them (e.g. with

Re: r213213 - DebugInfo: Forward HandleTagDeclRequiredDefinition through MultiplexConsumer to fix debug info emission in the presence of plugins.

2016-05-25 Thread Nico Weber via cfe-commits
Zombie review comment: ".test" isn't part of test/lit.cfg::config.suffixes, so the two .test files added in this change never run unless you explicitly run them (e.g. with `bin/llvm-lit ../llvm-rw/tools/clang/test/CodeGenCXX/*.test`). CodeGenCXX/debug-info-class-limited.test fails when I run it