[all-commits] [llvm/llvm-project] c4c0e7: [libcxx] Add _LIBCPP_NODEBUG to std::conditional r...
David Blaikie via All-commits
all-commits at lists.llvm.org
Tue Nov 22 15:38:18 PST 2022
Branch: refs/heads/main
Home: https://github.com/llvm/llvm-project
Commit: c4c0e7984ffdef8820eab3e66863e9a598d1d2ef
https://github.com/llvm/llvm-project/commit/c4c0e7984ffdef8820eab3e66863e9a598d1d2ef
Author: David Blaikie <dblaikie at gmail.com>
Date: 2022-11-22 (Tue, 22 Nov 2022)
Changed paths:
M libcxx/include/__type_traits/conditional.h
Log Message:
-----------
[libcxx] Add _LIBCPP_NODEBUG to std::conditional related typedfs
Looks like std::conditional wasn't included in 14d4869209cc8ff9a53aaa5a59d6409fbc1c5f5d
(& maybe other typedefs that should be using this technique either got
missed or have regressed since that change was made)
This was noticed by a 1.4% clang.dwp regression due to
f4fb72e6d4cee1097e6606e66232fe55e793cd86 introducing more instantiations
of std::conditional - this change reduces that regression to 0.6% at
least.
I'm also looking at other instantiations caused by that change that
might be able to be addressed - but a quick grep shows ~200 "type"
typedefs missing _LIBCPP_NODEBUG, so maybe a systematic application of
the typedef might be suitable?
Differential Revision: https://reviews.llvm.org/D131082
More information about the All-commits
mailing list