[llvm] [llvm][OpenMP][NFC] Cleanup AtomicInfo (PR #119199)
Krzysztof Parzyszek via llvm-commits
llvm-commits at lists.llvm.org
Tue Dec 10 04:21:54 PST 2024
kparzysz 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.
https://github.com/llvm/llvm-project/pull/119199
More information about the llvm-commits
mailing list