[PATCH] D105439: [clang] protects users from relying on libc++ detail headers

Christopher Di Bella via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 12 09:33:03 PDT 2021


cjdb added a comment.

In D105439#2871429 <https://reviews.llvm.org/D105439#2871429>, @Quuxplusone wrote:

> Given that the goal is to get `include-what-you-use` to stop suggesting that users `#include <__detail/fooimpl.h>` when what they want is spelled `<foo.h>`, I think what's desired here (if anything) would be a pragma targeted at `include-what-you-use` (and other tools). Basically
>
>   // in __detail/fooimpl.h
>   #pragma clang public_header "foo.h"
>
> This pragma can mostly be ignored by Clang; it's just up to those other tools to consume these pragmas and report the correct public header.

Thank you for your opinion. That is not the purpose of this patch.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D105439



More information about the cfe-commits mailing list