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

[Peter Rong via Phabricator via llvm-commits]

Peter 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.
----------------
arsenm wrote:
> 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 
This rule is now added to `isCompatibleReplacement`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139907