[PATCH] D139907: [FuzzMutate] RandomIRBuilder has more source and sink type now.

[Matt Arsenault via Phabricator via llvm-commits]

arsenm added inline comments.


================
Comment at: llvm/lib/FuzzMutate/RandomIRBuilder.cpp:284-286
+        // TODO: Replacing operands of intrinsics would be interesting,
+        // but there's no easy way to verify that a given replacement is
+        // valid given that intrinsics can impose arbitrary constraints.
----------------
Peter wrote:
> arsenm wrote:
> > Not really, you just need to avoid touching immarg parameters
> I am not sure how to determine if an operand of an intrinsic should be immarg or not, would you please elabrate?
Something like  F.hasParamAttr(i, Attribute::ImmArg) should work 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139907