[PATCH] D141134: [NFC] Only expose getXXXSize functions in TypeSize

Guillaume Chatelet via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jan 12 01:28:56 PST 2023


gchatelet closed this revision.
gchatelet added a comment.

So i've submitted the following patches which remove the usage of `TypeSize::getFixedSize()` and `TypeSize::getKnownMinSize()`

- rG6916ebd02650 <https://reviews.llvm.org/rG6916ebd026500061462917666a0e0d228ed52681>: [clang][NFC] Use the TypeSize::getXXXValue() instead of TypeSize::getXXXSize).Wed, Jan 11, 5:08 PM
- rGca8485578652 <https://reviews.llvm.org/rGca848557865272046901c1de1e7e9ba8c9cc3137>: [mlir][NFC] Use TypeSize::getFixedValue() instead of TypeSize::getFixedSize().Wed, Jan 11, 5:22 PM
- rGfbd207fd6f6b <https://reviews.llvm.org/rGfbd207fd6f6bc76a41cbf90fb6d87c6fa627ddf7>: [NFC] Use TypeSize::getKnownMinValue() instead of TypeSize::getKnownMinSize().Wed, Jan 11, 5:29 PM
- rG48f5d77eeed4 <https://reviews.llvm.org/rG48f5d77eeed4d7453d199c4b4ac16c1bbfbdbddf>: [NFC] Use TypeSize::getKnownMinValue() instead of TypeSize::getKnownMinSize().Wed, Jan 11, 5:37 PM
- rG8fd5558b2976 <https://reviews.llvm.org/rG8fd5558b29761aec91143129cb23c90581a944e9>: [NFC] Use TypeSize::geFixedValue() instead of TypeSize::getFixedSize().Wed, Jan 11, 5:49 PM

I've also marked the functions deprecated so downstream users can fix their codebase.

- rG1dcaa2bdf230 <https://reviews.llvm.org/rG1dcaa2bdf2300ad5bd28dcdf1b46a68ad1a95f5d>: Marking TypeSize getFixedSize() and getKnownMinSize() deprecated.

I'm marking this patch as closed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D141134



More information about the cfe-commits mailing list