[PATCH] D67052: Add reference type transformation builtins
Zoe Carver via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Sep 3 18:13:48 PDT 2019
zoecarver marked an inline comment as done.
zoecarver added inline comments.
================
Comment at: libcxx/test/libcxx/utilities/meta/stress_tests/stress_test_remove_reference.sh.cpp:13
+// variants listed in the RUN script.
+//
+// Impl Compile Time Object Size
----------------
zoecarver wrote:
> I think there is a good chance the reason that my added builtin isn't significantly faster is that `std::remove_reference` is already optimized somewhere. Thoughts?
I think if I updated this test to actually remove a reference, it might show more of a difference.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D67052/new/
https://reviews.llvm.org/D67052
More information about the cfe-commits
mailing list