search

banach-space / llvm-tutor

A collection of out-of-tree LLVM passes for teaching and learning
MIT License
2.95k stars 393 forks source link

Migrated to use Module::createRNG and removed anonymous namespace where static functions were appropriate #110

Closed qbojj closed 8 months ago

qbojj commented 8 months ago

Changed DuplicateBB to use Module::createRNG.

As LLVM style guide states:

make anonymous namespaces as small as possible, and only use them for class declarations.

but in fcmpEq plugins those were used for functions -> changed to use static.

banach-space commented 8 months ago

Nice, thank you!