Re: r361428 - Modules: Code generation of enum constants for merged enum definitions

2019-05-22 Thread David Blaikie via cfe-commits
Ah, sure - thanks for the heads up! Fixed in r361439 On Wed, May 22, 2019 at 2:49 PM Galina Kistanova wrote: > Hello David, > > This commit broke the test on the builder: > > http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/17793 > Please have a look? > . . . > Fai

Re: r361428 - Modules: Code generation of enum constants for merged enum definitions

2019-05-22 Thread Galina Kistanova via cfe-commits
Hello David, This commit broke the test on the builder: http://lab.llvm.org:8011/builders/llvm-clang-x86_64-expensive-checks-win/builds/17793 Please have a look? . . . Failing Tests (1): Clang :: Modules/enum-codegen.cpp The builder was already red and did not send any notifications. Thanks

r361428 - Modules: Code generation of enum constants for merged enum definitions

2019-05-22 Thread David Blaikie via cfe-commits
Author: dblaikie Date: Wed May 22 13:36:06 2019 New Revision: 361428 URL: http://llvm.org/viewvc/llvm-project?rev=361428&view=rev Log: Modules: Code generation of enum constants for merged enum definitions Found in a bootstrap of LLVM with implicit modules, resulting in a deadlock of some Orc uni