jrtc27 added inline comments.
================
Comment at: clang/lib/CodeGen/CGBuilder.h:158
/// This method is to be deprecated. Use `Address::withElementType` instead.
+ [[deprecated("Use `Address::withElementType` instead.")]]
----------------
JOE1994 wrote:
> nikic wrote:
> > JOE1994 wrote:
> > >
> > This is a private method, so simply delete it instead of deprecating.
> It seems like this method is listed as "public" member functions in
> https://clang.llvm.org/doxygen/classclang_1_1CodeGen_1_1CGBuilderTy.html .
>
> I see the `public` specifier on line 50.
It's public in the sense of its access specifier, i.e. that it can be used
outside of CGBuiltin, but it's private in the sense that this header is in
clang/lib/CodeGen and thus only used within Clang itself, not exposed as a
Clang API, so if Clang isn't using it any more, nothing is.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D154285/new/
https://reviews.llvm.org/D154285
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
