]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
Revert clang change that breaks CTF on aarch64
authorDimitry Andric <dim@FreeBSD.org>
Sun, 12 Dec 2021 20:11:40 +0000 (21:11 +0100)
committerDimitry Andric <dim@FreeBSD.org>
Wed, 15 Dec 2021 20:09:50 +0000 (21:09 +0100)
commitdefb7da9772ad5e007493bdb1d32a42c448b9c7f
tree6c09f1a924dc3f2d1b0149e4435674db6b302065
parent55e020a6f9fcbe80bfd8f532cae9c22cf4ebd74b
Revert clang change that breaks CTF on aarch64

Revert commit e655e74a318e from llvm git (by Peter Collingbourne):

  AST: Create __va_list in the std namespace even in C.

  This ensures that the mangled type names match between C and C++,
  which is significant when using -fsanitize=cfi-icall. Ideally we
  wouldn't have created this namespace at all, but it's now part of
  the ABI (e.g. in mangled names), so we can't change it.

  Differential Revision: https://reviews.llvm.org/D104830

As reported by Jessica in https://reviews.llvm.org/D104830#3129527, this
upstream change is implemented in such a way that it breaks DTrace's
CTF. Since a proper fix has not yet been forthcoming, and we are
unaffected by the (CFI-related) problem upstream was trying to address,
revert the change for now.

Requested by: jrtc27
MFC after: 3 days

(cherry picked from commit da2012af42fb704365cfaff7ae68fc7de59981da)
contrib/llvm-project/clang/lib/AST/ASTContext.cpp