[PATCH] D142708: [NFC] Transition GlobalObject alignment from MaybeAlign to Align

Guillaume Chatelet via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 31 07:09:27 PST 2023


gchatelet added a comment.

Breakage came from GCC being more strict on type redefinition

  /b/sanitizer-x86_64-linux/build/llvm-project/llvm/include/llvm/LTO/LTO.h:292:13: error: declaration of ‘llvm::Align llvm::lto::LTO::RegularLTOState::CommonResolution::Align’ changes meaning of ‘Align’ [-fpermissive]
    292 |       Align Align;
        |             ^~~~~

I changed the variable name to `Alignment` instead.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D142708/new/

https://reviews.llvm.org/D142708



More information about the llvm-commits mailing list