maxjhandsome
/
llvm-project
forked from OSchip/llvm-project
1
0
Fork 0
Code Issues Pull Requests Packages Releases Wiki Activity
llvm-project/libcxx
History
Dimitry Andric 1b1b80f45f Put type attributes after class keyword 
Summary:
Compiling `<functional>` in C++17 or higher mode results in:

```
functional:2500:1: warning: attribute '__visibility__' is ignored, place it after "class" to apply attribute to type declaration [-Wignored-attributes]
_LIBCPP_TYPE_VIS
^
__config:701:46: note: expanded from macro '_LIBCPP_TYPE_VIS'
#    define _LIBCPP_TYPE_VIS __attribute__ ((__visibility__("default")))
                                             ^
1 warning generated.
```

Fix it by putting the attribute after the `class` keyword.

Reviewers: EricWF, mclow.lists

Reviewed By: EricWF

Subscribers: cfe-commits

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

llvm-svn: 325027
 		2018-02-13 17:40:59 +00:00
..
benchmarks Update Google Benchmark library 2018-01-18 04:23:01 +00:00
cmake [cmake] Always respect existing CMAKE_REQUIRED_FLAGS when adding additional ones. 2018-01-23 03:30:23 +00:00
docs Fix typos. 2018-02-13 08:12:00 +00:00
fuzzing Wrote my own version of is_permutation; that was dominating the timings 2018-01-19 03:17:45 +00:00
include Put type attributes after class keyword 2018-02-13 17:40:59 +00:00
lib LLVM_FOUND isn't always set, so just test if llvm_setup_rpath() is 2018-01-27 18:55:30 +00:00
src Fix building libc++ with the macOS 10.13 SDK with -mmacosx-version-min=10.12 or lower. 2018-02-06 19:17:41 +00:00
test [libcxx] [test] Strip trailing whitespace, NFC. 2018-02-12 22:54:35 +00:00
utils Update Google Benchmark library 2018-01-18 04:23:01 +00:00
www Implement LWG 2835 - fix <tgmath.h> 2018-02-12 19:13:24 +00:00
.arcconfig [libcxx] Set up .arcconfig to point to new Diffusion CXX repository 2017-12-04 17:55:28 +00:00
.clang-format Make clang-format use C++03 syntax 2017-04-15 02:45:43 +00:00
.gitignore Implement N4606 optional 2016-10-12 07:46:20 +00:00
CMakeLists.txt Add default C++ ABI libname and include paths for FreeBSD 2018-02-11 22:31:05 +00:00
CREDITS.TXT Try again, this time with the correct address 2017-12-29 19:26:53 +00:00
LICENSE.TXT Update year to 2017 2017-01-03 11:20:43 +00:00
NOTES.TXT Update doc and various cleanup 2016-12-23 20:00:13 +00:00
TODO.TXT Add list of filesystem NB comments to TODO.TXT so they can be tracked separately 2017-03-06 21:23:36 +00:00
appveyor-reqs-install.cmd Update Appveyor LLVM install link to be current and non-broken 2017-11-15 01:37:11 +00:00
appveyor.yml Rename Appveyor install helper script. 2017-05-10 02:43:02 +00:00