[PATCH] D142384: [C++20] Fix a crash with modules.

Balázs Benics via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Mar 3 07:22:56 PST 2023


steakhal added a comment.

@usaxena95 @ilya-biryukov, @hans, I wonder if we should backport this change to `release/16.x`. Otherwise, clang-16 won't have this fix. WDYT?
I'm also a bit worried that we don't have tests. And that we have "unexpected" sideeffects like what you mentioned @hans.

Given all these, I would probably vote for not backporting this, but this is the chance if we want.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D142384



More information about the cfe-commits mailing list