arcana-lab / noelle

NOELLE Offers Empowering LLVM Extensions
MIT License
75 stars 36 forks source link

Add/Update include guards to headers #106

Closed kala-ghoda closed 10 months ago

kala-ghoda commented 10 months ago
kala-ghoda commented 10 months ago

Hi @tommymcm, thanks for assigning the task to me. I have created an initial PR just for ".h" header files. If this is according to the standards, I will proceed with ".hpp" headers. Thanks! 😄

tommymcm commented 10 months ago

Hey @kala-ghoda, the format of this is correct, but the include guards are not needed for any files under the test/ directory, only those under external/ (which are good the way you have them) and src/.

Thanks! Tommy

kala-ghoda commented 10 months ago

Thanks, pushing fresh commit.