[PATCH] D13427: RFC: faster isa<IntrinsicInst> (bugged tests?)

[escha via llvm-commits]

escha added a comment.

Unfortunately, it seems like we have a deeper problem here. Some tests seem to rely on the ability to handle unknown intrinsics, but large parts of LLVM assume that not_intrinsic == not an intrinsic, even though sometimes it can mean unknown_intrinsic. There's a ton of inconsistencies here, and it doesn't seem easy to fix. I tried adding an unknown_intrinsic option, but that just opened an entire other can of worms because lots of code assumed that nonzero intrinsic ID == real, known intrinsic...


Repository:
  rL LLVM

http://reviews.llvm.org/D13427