diff options
author | Roman Lebedev <lebedev.ri@gmail.com> | 2020-01-23 22:50:34 +0300 |
---|---|---|
committer | Roman Lebedev <lebedev.ri@gmail.com> | 2020-01-23 22:50:50 +0300 |
commit | b749af6a1ff45434011278c3ba765a41f0ee02f2 (patch) | |
tree | 90e1977701bfbe1a4d66af070bd07195554e4bc0 /llvm/lib/Bitcode/Writer/BitcodeWriter.cpp | |
parent | 5ffe6408ffb6ad8642598ab03de04d58b4980e81 (diff) | |
download | llvm-b749af6a1ff45434011278c3ba765a41f0ee02f2.zip llvm-b749af6a1ff45434011278c3ba765a41f0ee02f2.tar.gz llvm-b749af6a1ff45434011278c3ba765a41f0ee02f2.tar.bz2 |
[Sema] Don't disallow placing `__attribute__((alloc_align(param_idx)))` on `std::align_val_t`-typed parameters
Summary:
I kind-of understand why it is restricted to integer-typed arguments,
for general enum's the value passed is not nessesairly the alignment implied,
although one might say that user would know best.
But we clearly should whitelist `std::align_val_t`,
which is just a thin wrapper over `std::size_t`,
and is the C++ standard way of specifying alignment.
Reviewers: erichkeane, rsmith, aaron.ballman, jdoerfert
Reviewed By: erichkeane
Subscribers: cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D73019
Diffstat (limited to 'llvm/lib/Bitcode/Writer/BitcodeWriter.cpp')
0 files changed, 0 insertions, 0 deletions