[PATCH] D95488: Itanium Mangling: In 'enable_if', omit X/E around <expr-primary>.

James Y Knight via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Jan 26 15:49:13 PST 2021


jyknight created this revision.
jyknight added reviewers: rsmith, rjmccall.
jyknight requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.

The Clang enable_if extension is mangled as an <extended-qualifier>,
which is supposed to contain <template-args>. However, we were
unconditionally emitting X/E around its arguments, neglecting the fact
that <expr-primary> should be emitted directly without the surrounding
X/E.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D95488

Files:
  clang/lib/AST/ItaniumMangle.cpp
  clang/test/CodeGen/enable_if.c
  clang/test/CodeGenCXX/enable_if.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D95488.319424.patch
Type: text/x-patch
Size: 4688 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20210126/dfd06ed9/attachment.bin>


More information about the cfe-commits mailing list