[PATCH] D137458: [clang] Add __decay as a builtin template

2023-02-10 Thread Christopher Di Bella via Phabricator via cfe-commits
cjdb added subscribers: aaron.ballman, cjdb. cjdb added a comment. In D137458#3909343 , @troyj wrote: >> Also, it would be nice to have some numbers for the 'measurably faster' >> claim :) > > Sure. Here's an example of a library change that started usin

[PATCH] D137458: [clang] Add __decay as a builtin template

2022-11-04 Thread Troy Johnson via Phabricator via cfe-commits
troyj added a comment. > Also, it would be nice to have some numbers for the 'measurably faster' claim > :) Sure. Here's an example of a library change that started using builtins for `__make_integer_seq` and `__type_pack_element` https://github.com/facebook/fatal/commit/58102a3f7e66ad122d7d33

[PATCH] D137458: [clang] Add __decay as a builtin template

2022-11-04 Thread Roy Jacobson via Phabricator via cfe-commits
royjacobson added a comment. I worry about this mangling issue https://github.com/llvm/llvm-project/issues/54993. For std::make_index_sequence that wasn't that big of a deal, since nobody really uses that as return type of (template) functions, but my guess is that enough people do things like

[PATCH] D137458: [clang] Add __decay as a builtin template

2022-11-04 Thread Troy Johnson via Phabricator via cfe-commits
troyj created this revision. troyj added a reviewer: bruno. Herald added a subscriber: martong. Herald added a reviewer: shafik. Herald added a project: All. troyj requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. `__decay(T)` exists as a bui