[libcxx-commits] [PATCH] D82627: Fix CFI issues in <future>

Evgenii Stepanov via Phabricator via libcxx-commits libcxx-commits at lists.llvm.org
Mon Jul 20 21:16:27 PDT 2020


eugenis added subscribers: pcc, kcc.
eugenis added a comment.

They are "errors reported by Control Flow Integrity" - that part is true. This is valid C++, but it triggers false positive errors in CFI.
This is not without precedent - see https://github.com/llvm/llvm-project/commit/3e58a6a7, https://github.com/llvm/llvm-project/commit/f11c00d7 and more.
The fact that CFI is stricter that the wording of the standard has been discussed many times before, but the conclusion is always that these reports are useful.
This type of false positive is not common at all outside of libc++.
@pcc 
@kcc


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82627





More information about the libcxx-commits mailing list