Page MenuHomePhabricator

Compiler.h: remove old GCC checks, update docs
Needs ReviewPublic

Authored by jfb on Aug 14 2019, 12:52 PM.
This revision needs review, but there are no reviewers specified.

Details

Reviewers
None
Summary

We can avoid checking for older GCC versions now that the minimum is 5.1. While I'm there, update doxygen comments for most macros.

Event Timeline

jfb created this revision.Aug 14 2019, 12:52 PM
xbolva00 added inline comments.
jfb marked 2 inline comments as done.Aug 15 2019, 10:36 AM
jfb added inline comments.
llvm/include/llvm/Support/Compiler.h
356

The comment is, but it's still true: it was added in llvm.org/r217349 from September 7th 2014 and the minimum clang supported right now is 3.5, released on September 3rd 2014. I'll update the comment, because the check _has_builtin(__builtin_assume_aligned) is sufficient.