[PATCH] D152570: [clang] Apply -fmacro-prefix-map to anonymous tags in template arguments

Fangrui Song via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jun 13 16:24:17 PDT 2023


MaskRay added a comment.

In D152570#4417752 <https://reviews.llvm.org/D152570#4417752>, @dankm wrote:

> @MaskRay if this meets your approval, are you able to commit it for me?

LG, but give @aaron.ballman and @shafik some time if they have opinions.

In D152570#4416201 <https://reviews.llvm.org/D152570#4416201>, @dankm wrote:

> Maybe for the description something like:
>
>   Apply -fmacro-prefix-map to anonymous tags in template arguments
>
> Do I need to mention TypePrinter?

I think it's fine to not mention it in the subject. You can mention TypePrinter in the description, though.



================
Comment at: clang/docs/ReleaseNotes.rst:500
   (`#50244 <https://github.com/llvm/llvm-project/issues/50244>_`).
+- Fix lambdas in template arguments not respecting ``-fmacro-prefix-map``
+  (`#63219 <https://github.com/llvm/llvm-project/issues/63219>`_)
----------------
This needs adjustment as the subject has changed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D152570



More information about the cfe-commits mailing list