[PATCH] D138339: [FuzzMutate] New strategy `ShuffleBlockStrategy`

Peter Rong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 28 16:32:06 PST 2022


Peter added inline comments.


================
Comment at: llvm/unittests/FuzzMutate/StrategiesTest.cpp:396
+      %5 = alloca i32, align 4 \n\
+      store i32 %0, i32* %3, align 4 \n\
+      store i32 %1, i32* %4, align 4 \n\
----------------
arsenm wrote:
> Peter wrote:
> > arsenm wrote:
> > > Use opaque pointers
> > No unit testing in this folder is supporting opaque pointers yet. 
> > I guess we have to change how the unit tests are run to support that. 
> > I can do it in another patch if you can enlighten me how.
> Shouldn't need to change anything other than the pointer next. s/i32*/ptr/ 
directly changing `i32*` into `ptr` will get an error when running parser and abort the unit test. ("warning: ptr type is only supported in -opaque-pointers mode")


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D138339/new/

https://reviews.llvm.org/D138339



More information about the llvm-commits mailing list