morehouse added inline comments.
================ Comment at: compiler-rt/lib/fuzzer/FuzzerInterceptors.cpp:17 + +#define GET_CALLER_PC() __builtin_return_address(0) + ---------------- Nit: Let's move this down with the other defines. ================ Comment at: compiler-rt/lib/fuzzer/FuzzerInterceptors.cpp:30 + +#include <dlfcn.h> // for dlsym() + ---------------- Nit: Can we move the other includes down by this one? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D83494/new/ https://reviews.llvm.org/D83494 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
