[llvm] [llvm][OpenMP][NFC] Cleanup AtomicInfo (PR #119199)

via llvm-commits llvm-commits at lists.llvm.org
Sun Dec 22 20:30:43 PST 2024


NimishMishra wrote:

> Please fix the comments at the top of both files. Both say "Atomic.h", and the first line is broken into two.
> 
> I think that too many includes are now removed. For example, the header file uses `IntegerType::get`, but there is no `#include "llvm/IR/DerivedTypes.h"`, or `#include <utility>` for `std::pair`. I believe the preference is to IWYU, but I'll defer to @nikic to decide.

Thanks. I have made the changes.

https://github.com/llvm/llvm-project/pull/119199


More information about the llvm-commits mailing list