[PATCH] D126959: [C++20][Modules] Introduce an implementation module.

Bjorn Pettersson via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 27 01:18:28 PDT 2023


bjope added a comment.

In D126959#4222854 <https://reviews.llvm.org/D126959#4222854>, @iains wrote:

> In D126959#4222637 <https://reviews.llvm.org/D126959#4222637>, @bjope wrote:
>
>> This seem to case problems when building with asan enabled (LLVM_USE_SANITIZER='Address'):
>
> investigating... do you need the patch reverted?

I've reverted it downstream right now, so personally I don't have a problem with this right now. Still, unless it will go quickly to find a fix it might help others people if reverting (it was not obvious to me which patch that caused the failures, and others might still be bisecting trying to find which commit that caused the trouble).


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D126959



More information about the cfe-commits mailing list