[PATCH] D84147: Use typedef to represent storage type in FPOption and FPOptionsOverride

Serge Pavlov via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jul 20 03:20:07 PDT 2020


sepavloff created this revision.
sepavloff added reviewers: rjmccall, mibintc.
Herald added a project: clang.

Size of FPOption is now 14 bit, which is closed to the current limit
of 16 bits. Adding new properties to FPOption would require change of
the types, which represent storage of FPOption and FPOptionsOverride.
To facilitate this change, the storage types were changed from standard
integer types to typedefs defined inside the proper classes. Checks for
size of these storage types were added.

No functional changes intended.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D84147

Files:
  clang/include/clang/Basic/FPOptions.def
  clang/include/clang/Basic/LangOptions.h
  clang/include/clang/Sema/Sema.h
  clang/include/clang/Serialization/ASTReader.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D84147.279167.patch
Type: text/x-patch
Size: 6837 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20200720/5f355313/attachment.bin>


More information about the cfe-commits mailing list