[PATCH] D53040: [libFuzzer] Generalize the code for getting the previous offset for different architectures

[George Karpenkov via Phabricator via llvm-commits]

george.karpenkov added inline comments.


================
Comment at: compiler-rt/lib/fuzzer/FuzzerTracePC.cpp:221
+  return PC + 1;
+#endif
+}
----------------
morehouse wrote:
> Could we avoid putting ifdefs here?  Maybe they could go in FuzzerDefs.h.
I don't think this is a good idea.

1. FuzzerDefs contains constants, this is not a constant.
2. As the code is cloned, it would be harder to maintain parity with the code in sanitizer_common if it's significantly changed.


https://reviews.llvm.org/D53040