[PATCH] D134653: [clang][modules] Over-align the `Module` class
Jan Svoboda via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Sep 26 09:40:46 PDT 2022
jansvoboda11 created this revision.
jansvoboda11 added reviewers: srj, mgorny, Bigcheese.
Herald added a subscriber: ributzka.
Herald added a project: All.
jansvoboda11 requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
This makes `llvm::PointerIntPair<Module *, 3>` from f35230ae <https://reviews.llvm.org/rGf35230ae0a69bbdd74a4faa3ae0b69a46796dc12> work across platforms.
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D134653
Files:
clang/include/clang/Basic/Module.h
Index: clang/include/clang/Basic/Module.h
===================================================================
--- clang/include/clang/Basic/Module.h
+++ clang/include/clang/Basic/Module.h
@@ -93,7 +93,7 @@
};
/// Describes a module or submodule.
-class Module {
+class alignas(8) Module {
public:
/// The name of this module.
std::string Name;
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134653.462947.patch
Type: text/x-patch
Size: 355 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220926/aa500bc2/attachment.bin>
More information about the cfe-commits
mailing list